Wireshark-dev: [Wireshark-dev] [PATCH] NDMP config_get_auth_attr_reply missing a field
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: "Aaron Christensen" <aaronmf@xxxxxxxxx>
Date: Thu, 29 Jun 2006 23:24:34 -0500
Added the dissection of a field that wasn't being picked up during NDMP MD5 authentication.

The NDMP_CONFIG_GET_AUTH_ATTR message response was using the function dissect_auth_attr_msg() which dissects an auth_attr struct.  However, the specification includes and error field previous to that struct.  Since that function was being called directly, the error field was being picked up as  part of the auth_attr, and then leaving some extra data at the end.

I added a new function, dissect_ndmp_config_get_auth_attr_reply(), which first calls dissect_error() and then dissect_auth_attr_msg().  I didn't  change dissect_auth_attr_msg() because the NDMP_CONNECT_SERVER_AUTH request uses it directly.  After testing it, it appears to be picking things up correctly, now.

Regards,
~Aaron Christensen

Attachment: ndmp_get_auth_attr.diff
Description: Binary data