diff options
author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2002-11-28 21:03:36 +0000 |
---|---|---|
committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2002-11-28 21:03:36 +0000 |
commit | 79c6e81d6df9352e4b4b3e285597b6b742d79f1e (patch) | |
tree | 68aa4f0b3ddf8c5b371242eaee8176a84b2e8445 /packet-dcerpc-spoolss.c | |
parent | df881acd93b45f9802486d61afb4f59d93f899ee (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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6699 f5534014-38df-0310-8fa8-9805f1628bb7
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; } |