diff options
| author | Jeremy Allison <jra@samba.org> | 2001-04-28 00:05:11 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2001-04-28 00:05:11 +0000 |
| commit | b238f4a2ffeb225d16857b594355dc60c976e3e0 (patch) | |
| tree | 27468e3b57527992947d4e4f56fb1cc085816fec /source3 | |
| parent | 8f11c24b74ff943e67a456c17c1ac139fa801c6d (diff) | |
| download | samba-b238f4a2ffeb225d16857b594355dc60c976e3e0.tar.gz samba-b238f4a2ffeb225d16857b594355dc60c976e3e0.tar.bz2 samba-b238f4a2ffeb225d16857b594355dc60c976e3e0.zip | |
rpcclient/rpcclient.c: Non-void return in void function.
smbd/sec_ctx.c: Fixed potential memory leak spotted by
Kenichi Okuyama@Tokyo Research Lab, IBM-Japan, Co.
utils/nmblookup.c: gcc warning on Solaris fix.
Jeremy.
(This used to be commit 1be60597cd62af2b30a1496d06ea5704e87d3b7d)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/smbd/sec_ctx.c | 4 | ||||
| -rw-r--r-- | source3/utils/nmblookup.c | 2 |
2 files changed, 4 insertions, 2 deletions
diff --git a/source3/smbd/sec_ctx.c b/source3/smbd/sec_ctx.c index f6563ffe96..6f98e9df6b 100644 --- a/source3/smbd/sec_ctx.c +++ b/source3/smbd/sec_ctx.c @@ -152,8 +152,10 @@ int get_current_groups(int *p_ngroups, gid_t **p_groups) return -1; } - if ((ngroups = sys_getgroups(ngroups,groups)) == -1) + if ((ngroups = sys_getgroups(ngroups,groups)) == -1) { + safe_free(groups); return -1; + } (*p_ngroups) = ngroups; (*p_groups) = groups; diff --git a/source3/utils/nmblookup.c b/source3/utils/nmblookup.c index 4e85f6ac45..0f978a6cf3 100644 --- a/source3/utils/nmblookup.c +++ b/source3/utils/nmblookup.c @@ -118,7 +118,7 @@ static void do_node_status(int fd, char *name, int type, struct in_addr ip) for (i=0;i<count;i++) { fstrcpy(cleanname, status[i].name); for (j=0;cleanname[j];j++) { - if (!isprint(cleanname[j])) cleanname[j] = '.'; + if (!isprint((int)cleanname[j])) cleanname[j] = '.'; } printf("\t%-15s <%02x> - %s\n", cleanname,status[i].type, |