Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 5790] Add Partial Flag and fix clang warning for Mongo Dissector

[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5790

--- Comment #6 from Alexis La Goutte <alexis.lagoutte@xxxxxxxxx> 2011-03-30 01:05:56 PDT ---
(In reply to comment #5)
> (In reply to comment #2)
> > Created an attachment (id=6093)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=6093) [details]
[details]
> > Mongo dissector : Fix clang Warning
> 
> What was the clang warning this patch supposedly fixes?  It looks like your
> changes just causes the mongo dissector to directly add the unknown data to the
> mongo tree instead of calling the "data" dissector to do it.  Did you submit
> the right patch?

Hi, it is the right patch but i change also the unknown data dissect

The part of code is fix the warning is :

+        if(offset < tvb_reported_length(tvb))
+        {
+            expert_add_info_format(pinfo, ti_len, PI_UNDECODED, PI_WARN, "
Unknown Data (not interpreted)");
+

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.