diff options
| author | Jeremy Allison <jra@samba.org> | 2006-06-28 04:30:51 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:19:03 -0500 |
| commit | 7d7a98208b1b514da60486fcbbb7b6d04df4ffac (patch) | |
| tree | 7bc451aefb6784cb17bf7c3136eeb313c9feba08 /source | |
| parent | d1a1c4e092877a6ea0f98eed2a37a96d42c36323 (diff) | |
| download | samba-7d7a98208b1b514da60486fcbbb7b6d04df4ffac.tar.gz samba-7d7a98208b1b514da60486fcbbb7b6d04df4ffac.tar.xz samba-7d7a98208b1b514da60486fcbbb7b6d04df4ffac.zip | |
r16608: Klocwork #2035. Null deref (bit of a stretch... :-).
Jeremy.
Diffstat (limited to 'source')
| -rw-r--r-- | source/client/smbspool.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source/client/smbspool.c b/source/client/smbspool.c index e5cd4f2e8e6..c9a7fbe10ef 100644 --- a/source/client/smbspool.c +++ b/source/client/smbspool.c @@ -72,6 +72,7 @@ static int smb_print(struct cli_state *, char *, FILE *); struct cli_state *cli; /* SMB interface */ char null_str[1]; int tries = 0; + const char *dev_uri; null_str[0] = '\0'; @@ -132,8 +133,9 @@ static int smb_print(struct cli_state *, char *, FILE *); * Find the URI... */ - if (getenv("DEVICE_URI") != NULL) - strncpy(uri, getenv("DEVICE_URI"), sizeof(uri) - 1); + dev_uri = getenv("DEVICE_URI"); + if (dev_uri) + strncpy(uri, dev_uri, sizeof(uri) - 1); else if (strncmp(argv[0], "smb://", 6) == 0) strncpy(uri, argv[0], sizeof(uri) - 1); else |