diff options
author | Guy Harris <guy@alum.mit.edu> | 2002-11-28 21:03:36 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2002-11-28 21:03:36 +0000 |
commit | 0e050b1107790bf4774e9059e6639f6706c5291b (patch) | |
tree | 68aa4f0b3ddf8c5b371242eaee8176a84b2e8445 /packet-dcerpc-spoolss.c | |
parent | ca642e12bacebdbc643ef95da07e7795534cefc6 (diff) |
Get rid of the "pdata" argument to "dissect_dcerpc_uint16s()" and
"dissect_ndr_uint16s()"; "dissect_ndr_uint16s()" is always passed a null
pointer, "dissect_dcerpc_uint16s()" is only called by
"dissect_ndr_uint16s()", and the pointer returned through "pdata" is
*NOT* guaranteed to be aligned on a 16-bit boundary so we don't want to
tempt people to blithely dereference that pointer.
svn path=/trunk/; revision=6699
Diffstat (limited to 'packet-dcerpc-spoolss.c')
-rw-r--r-- | packet-dcerpc-spoolss.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/packet-dcerpc-spoolss.c b/packet-dcerpc-spoolss.c index 5e17e4037b..66fc36c59e 100644 --- a/packet-dcerpc-spoolss.c +++ b/packet-dcerpc-spoolss.c @@ -2,7 +2,7 @@ * Routines for SMB \PIPE\spoolss packet disassembly * Copyright 2001-2002, Tim Potter <tpot@samba.org> * - * $Id: packet-dcerpc-spoolss.c,v 1.62 2002/11/28 04:56:43 guy Exp $ + * $Id: packet-dcerpc-spoolss.c,v 1.63 2002/11/28 21:03:36 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@ethereal.com> @@ -4323,7 +4323,7 @@ static int dissect_form_name(tvbuff_t *tvb, int offset, packet_info *pinfo, offset = dissect_ndr_uint16s( tvb, offset, pinfo, tree, drep, - hf_spoolss_form_name, len, NULL); + hf_spoolss_form_name, len); return offset; } @@ -5833,7 +5833,7 @@ dissect_notify_info_data_buffer(tvbuff_t *tvb, int offset, packet_info *pinfo, offset = dissect_ndr_uint16s( tvb, offset, pinfo, tree, drep, - hf_spoolss_notify_info_data_buffer_data, len, NULL); + hf_spoolss_notify_info_data_buffer_data, len); return offset; } |