summaryrefslogtreecommitdiff
path: root/source3/libsmb
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2007-11-19 17:43:28 -0800
committerJeremy Allison <jra@samba.org>2007-11-19 17:43:28 -0800
commitcae4c742a896c6adb02b3e7a3c091d02aebb7339 (patch)
tree18c7367a32cb1f011d1b7694a00f8624907d6dbf /source3/libsmb
parent428e663100fa2c009b16941e2a390587fdfc00b5 (diff)
downloadsamba-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')
-rw-r--r--source3/libsmb/clirap.c7
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 */