[LU-14893] 'lctl --device scratch-MDT0000 changelog_deregister' seg faults with no changelog user Created: 28/Jul/21  Updated: 19/Jun/23  Resolved: 10/Aug/21

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.14.0, Lustre 2.15.0
Fix Version/s: Lustre 2.15.0

Type: Bug Priority: Minor
Reporter: James Nunez (Inactive) Assignee: Emoly Liu
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Duplicate
is duplicated by LU-15310 sanity test_160g: mds1: User cl7 not ... Resolved
Related
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

When I run ‘lctl --device scratch-MDT0000 changelog_deregister’ on the MDS, I get

# lctl --device scratch-MDT0000 changelog_deregister
Segmentation fault

From dmesg:

[ 2499.711304] lctl[20902]: segfault at 0 ip 00007fa7663a1fe7 sp 00007ffdb6619bb0 error 4 in libc-2.17.so[7fa766367000+1c4000]

If you look at the usage output when using --help, the message indicates that the changelog user is optional.

# lctl --device scratch-MDT0000 changelog_deregister --help
deregister an existing changelog user
usage: --device <mdtname> changelog_deregister [<id>|cl<id>...]
                                                 [--help|-h]
                                                 [--user|-u <username>]

run <command> after connecting to device <devno>
--device <devno> <command [args ...]>

If the changelog user is a required parameter for ‘lctl changelog_deregister’, we need to change the usage for message for ‘lctl changelog_deregister’ to make the changelog user required and, if at all possible, not seg fault when a changelog user is not specified.



 Comments   
Comment by Gerrit Updater [ 30/Jul/21 ]

Emoly Liu (emoly@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/44432
Subject: LU-14893 lctl: check user for changelog_deregister
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e223157c93f5025403b4be273ca8f5fef2f2c181

Comment by Gerrit Updater [ 10/Aug/21 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44432/
Subject: LU-14893 lctl: check user for changelog_deregister
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 644cb83921451316d226d8de1c81a82abb4b333e

Comment by Peter Jones [ 10/Aug/21 ]

Landed for 2.15

Generated at Sat Feb 10 03:13:40 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.