diff options
| author | Jeremy Allison <jra@samba.org> | 2007-11-19 17:43:28 -0800 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2007-11-19 17:43:28 -0800 |
| commit | cae4c742a896c6adb02b3e7a3c091d02aebb7339 (patch) | |
| tree | 18c7367a32cb1f011d1b7694a00f8624907d6dbf /source3/libsmb/clirap.c | |
| parent | 428e663100fa2c009b16941e2a390587fdfc00b5 (diff) | |
| download | samba-cae4c742a896c6adb02b3e7a3c091d02aebb7339.tar.gz samba-cae4c742a896c6adb02b3e7a3c091d02aebb7339.tar.bz2 samba-cae4c742a896c6adb02b3e7a3c091d02aebb7339.zip | |
Always check return from push_ascii.
Jeremy.
(This used to be commit 9c3d10521e691169cfbb8b728f123911c3c970ae)
Diffstat (limited to 'source3/libsmb/clirap.c')
| -rw-r--r-- | source3/libsmb/clirap.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/source3/libsmb/clirap.c b/source3/libsmb/clirap.c index 9d106d0394..848a8d5482 100644 --- a/source3/libsmb/clirap.c +++ b/source3/libsmb/clirap.c @@ -216,6 +216,7 @@ bool cli_NetServerEnum(struct cli_state *cli, char *workgroup, uint32 stype, pstring param; int uLevel = 1; int count = -1; + size_t len; errno = 0; /* reset */ @@ -235,7 +236,11 @@ bool cli_NetServerEnum(struct cli_state *cli, char *workgroup, uint32 stype, SIVAL(p,0,stype); p += 4; - p += push_ascii(p, workgroup, sizeof(pstring)-PTR_DIFF(p,param)-1, STR_TERMINATE|STR_UPPER); + len = push_ascii(p, workgroup, sizeof(pstring)-PTR_DIFF(p,param)-1, STR_TERMINATE|STR_UPPER); + if (len == (size_t)-1) { + return false; + } + p += len; if (cli_api(cli, param, PTR_DIFF(p,param), 8, /* params, length, max */ |