Wireshark-bugs: [Wireshark-bugs] [Bug 10636] New: crashing while dissecting nfs traces with call

Date: Tue, 28 Oct 2014 00:31:04 +0000
Bug ID 10636
Summary crashing while dissecting nfs traces with callbacks
Product Wireshark
Version Git
Hardware x86
OS All
Status UNCONFIRMED
Severity Major
Priority Low
Component Dissection engine (libwireshark)
Assignee [email protected]
Reporter [email protected]

Created attachment 13208 [details]
[PATCH] rpc: ignore duplicate program registrations

Build Information:
local build of 58304b9e3e2b "S1AP: Add an S1AP context to be able to determine
which message and..."
--
4afd70d4e46c "Use g_hash_table_new_full to free some values" (fix for bug
10261) introduced a use-after-free.

The problem is that some rpc_call->proc_info can point to an rpc procedure, so
it isn't necessarily safe to free rpc procedures.

Results are segfaults and garbled text in the info column for NFSv4 callbacks.

The attached patch fixes the segfaults and garbled text, by just refusing to
register duplicate rpc procedures.  (They are never removed from the hash
table, so the only time they are freed is when they're replaced.)

Another alternative might be just to revert the original change.


You are receiving this mail because:
  • You are watching all bug changes.