diff options
| author | Steve French <stevef@smf-t60p.smfdom> | 2008-07-23 14:25:17 -0500 |
|---|---|---|
| committer | Steve French <stevef@smf-t60p.smfdom> | 2008-07-23 14:25:17 -0500 |
| commit | 224f9b839bb5e776ead7d93e8e17b48846f5caf5 (patch) | |
| tree | d58114a49a5ab6a506a7b8ccc272f11ea6ec2fb4 /source3 | |
| parent | 7014c97aab8d252fce6c0e6a9be06e5181ec8f0e (diff) | |
| download | samba-224f9b839bb5e776ead7d93e8e17b48846f5caf5.tar.gz samba-224f9b839bb5e776ead7d93e8e17b48846f5caf5.tar.bz2 samba-224f9b839bb5e776ead7d93e8e17b48846f5caf5.zip | |
cifs.upcall: fix compile warning
Steve French noticed these warnings when building cifs.upcall:
Compiling client/cifs.upcall.c
client/cifs.upcall.c: In function 'usage':
client/cifs.upcall.c:204: warning: declaration of 'prog' shadows a global declaration
client/cifs.upcall.c:33: warning: shadowed declaration is here
Change the usage function to not take and arg and have it just use the global
"prog" variable. Fix a typo in the log message generated when an unknown
option is specified. Also getopt() always returns '?' when it sees an unknown
option so there's no point in printing it out.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(This used to be commit 4b8362f9777debd21c47154e786b7017fbf847f7)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/client/cifs.upcall.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/source3/client/cifs.upcall.c b/source3/client/cifs.upcall.c index 3860f33e38..e3893260dd 100644 --- a/source3/client/cifs.upcall.c +++ b/source3/client/cifs.upcall.c @@ -201,7 +201,7 @@ int cifs_resolver(const key_serial_t key, const char *key_descr) } void -usage(const char *prog) +usage() { syslog(LOG_WARNING, "Usage: %s [-c] [-v] key_serial", prog); fprintf(stderr, "Usage: %s [-c] [-v] key_serial\n", prog); @@ -234,7 +234,7 @@ int main(const int argc, char *const argv[]) goto out; } default:{ - syslog(LOG_WARNING, "unknow option: %c", c); + syslog(LOG_WARNING, "unknown option: %c", c); goto out; } } @@ -242,7 +242,7 @@ int main(const int argc, char *const argv[]) /* is there a key? */ if (argc <= optind) { - usage(prog); + usage(); goto out; } |