diff options
| author | Volker Lendecke <vl@samba.org> | 2018-08-08 10:14:26 +0200 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2018-08-23 10:38:28 +0200 |
| commit | 0eaf84a4f5c313776c6feab2968f241b5ce55a91 (patch) | |
| tree | 669e7934d304a144ccddb5abd465792c7eb19820 | |
| parent | b1b882c2c459b206e2455f53b3926ff9a13f7335 (diff) | |
| download | samba-0eaf84a4f5c313776c6feab2968f241b5ce55a91.tar.gz | |
libsmb: Fix CID 1438243 Unchecked return value
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13553
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Aug 8 23:10:22 CEST 2018 on sn-devel-144
| -rw-r--r-- | source3/libsmb/cliquota.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/source3/libsmb/cliquota.c b/source3/libsmb/cliquota.c index 8efd2bbe38a..52f98eb9e8f 100644 --- a/source3/libsmb/cliquota.c +++ b/source3/libsmb/cliquota.c @@ -649,7 +649,14 @@ NTSTATUS fill_quota_buffer(TALLOC_CTX *mem_ctx, /* pidl will align to 8 bytes due to 8 byte members*/ /* Remember how much align padding we've used. */ padding = qndr->offset; - ndr_push_align(qndr, 8); + + err = ndr_push_align(qndr, 8); + if (!NDR_ERR_CODE_IS_SUCCESS(err)) { + DBG_DEBUG("ndr_push_align returned %s\n", + ndr_map_error2string(err)); + return ndr_map_error2ntstatus(err); + } + padding = qndr->offset - padding; /* |