diff options
| author | Stefan Metzmacher <metze@samba.org> | 2011-11-18 13:19:19 +0100 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2011-11-18 13:30:48 +0100 |
| commit | d3cb61cf05485eda26280186bfa3850e2e6bcca9 (patch) | |
| tree | e112c4e8ed7a43f8532506bed3f2af88d23d7742 /source3/libsmb/clitrans.c | |
| parent | 48bcb8c846532ccd5489ef705182fb81c5278b04 (diff) | |
| download | samba-d3cb61cf05485eda26280186bfa3850e2e6bcca9.tar.gz samba-d3cb61cf05485eda26280186bfa3850e2e6bcca9.tar.bz2 samba-d3cb61cf05485eda26280186bfa3850e2e6bcca9.zip | |
s3:libsmb: fix compiler warning in cli_pull_trans()
metze
Diffstat (limited to 'source3/libsmb/clitrans.c')
| -rw-r--r-- | source3/libsmb/clitrans.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source3/libsmb/clitrans.c b/source3/libsmb/clitrans.c index 329ddcaac6..8ac31d89f1 100644 --- a/source3/libsmb/clitrans.c +++ b/source3/libsmb/clitrans.c @@ -85,6 +85,7 @@ static NTSTATUS cli_pull_trans(uint8_t *inbuf, uint32_t *pdata_disp, uint8_t **pdata) { uint32_t param_ofs, data_ofs; + uint8_t expected_num_setup; if (expect_first_reply) { if ((wct != 0) || (num_bytes != 0)) { @@ -99,6 +100,7 @@ static NTSTATUS cli_pull_trans(uint8_t *inbuf, if (wct < 10) { return NT_STATUS_INVALID_NETWORK_RESPONSE; } + expected_num_setup = wct - 10; *ptotal_param = SVAL(vwv + 0, 0); *ptotal_data = SVAL(vwv + 1, 0); *pnum_param = SVAL(vwv + 3, 0); @@ -108,7 +110,7 @@ static NTSTATUS cli_pull_trans(uint8_t *inbuf, data_ofs = SVAL(vwv + 7, 0); *pdata_disp = SVAL(vwv + 8, 0); *pnum_setup = CVAL(vwv + 9, 0); - if (wct < 10 + (*pnum_setup)) { + if (expected_num_setup < (*pnum_setup)) { return NT_STATUS_INVALID_NETWORK_RESPONSE; } *psetup = vwv + 10; |