OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_74_1-187-ga2f3be5
Gerrit Code Review
gerrit@openafs.org
Thu, 3 Jun 2010 10:58:53 -0700 (PDT)
The following commit has been merged in the master branch:
commit a2f3be5265a9e8a7f2cf0dd751a9ab82d289cef8
Author: Andrew Deason <adeason@sinenomine.net>
Date: Thu Jun 3 09:54:28 2010 -0500
up: refuse multicharacter arguments
The 'up' command currently silently accepts and discards extra
characters when specifying arguments. This can produce rather
confusing behavior such as mistyping '-v -1' as '-v-1' resulting in
the '-v' switch being honored, but the '-1' being ignored. The same
thing occurs for specifying '-v1', even though the usage message
implies that you can combine arguments.
So instead, report an error message for any arguments specified that
are longer than 2 characters, since they are never valid.
Change-Id: I64846b53248ea1d06d03b6ac1fdb4317ba04b03b
Reviewed-on: http://gerrit.openafs.org/2073
Tested-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/venus/up.c | 9 ++++++++-
1 files changed, 8 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository