summaryrefslogtreecommitdiff
path: root/udev/udevadm-test.c
diff options
context:
space:
mode:
authorSergey Vlasov <vsu@altlinux.ru>2008-11-15 00:34:43 +0300
committerKay Sievers <kay.sievers@vrfy.org>2008-11-15 02:01:15 +0100
commit56116314d114fb7e8a9c7ce6dc3ef4e929f9ecb0 (patch)
tree922b15ad0d2d8bafbb023db4c2bc1c30e68ca932 /udev/udevadm-test.c
parentf454ecf7544077349c24e9fcfd418dbfca927063 (diff)
udevadm: fix option parsing breakage with klibc
The klibc implementation of getopt_long() behaves slightly different from the glibc one - in particular, it treats the change of the option string argument between invocations as start of parsing a different command line, and resets its state. However, the udevadm code expected getopt_long() invocations in subcommands to continue parsing the rest of command line after initial options has been parsed at the top level; with klibc this broke, causing all udevadm subcommands to stop recognizing their options. Instead of relying on the glibc behavior, reset the getopt_long() state properly before invoking the subcommand handler: move argv to point to the subcommand name, decrease argc appropriately, and set optind = 0. This also fixes a minor bug visible with glibc - without setting optind = 0 all getopt_long() calls in subcommand handlers were behaving as if "+" was specified as the first character of the option string (which disables option reordering), because that state was set by the first getopt_long() call at the top level, and was not reset when parsing subcommand options. Signed-off-by: Sergey Vlasov <vsu@altlinux.ru>
Diffstat (limited to 'udev/udevadm-test.c')
0 files changed, 0 insertions, 0 deletions