Wireshark mailing list archives
Re: removing mergecap -T option
From: Pascal Quantin <pascal.quantin () gmail com>
Date: Fri, 14 Aug 2015 19:23:12 +0200
Le 14 août 2015 6:30 PM, "Hadriel Kaplan" <the.real.hadriel () gmail com> a écrit :
But isn't that the purpose of editcap? I'm only talking about removing it from mergecap.
Mmm, sorry I did not realize that. No specific objection on my side then. Pascal.
-hadriel On Fri, Aug 14, 2015 at 12:26 PM, Pascal Quantin <pascal.quantin () gmail com> wrote:Le 14 août 2015 6:18 PM, "Hadriel Kaplan" <the.real.hadriel () gmail com> a écrit :Howdy, Due to some reported bugs and inconsistencies, I'm refactoring the capture file merging code in mergecap.c and file.c's cf_merge_files() - basically gutting them and putting most of the logic into a common "merge_files()" function in merge.c. While merging their function code logic, I noticed that mergecap lets the user set the frame encap type to use for the new output file, with a '-T' option. That seems completely useless to me, since you can't really change the encap type of what was in the original capture files; nor does mergecap seem to do so either, nor does it filter/save only those packet encap types, nor verify that the input files had that, nor really anything useful - all it appears it can accomplish is to produce invalid output files. Does anyone mind if I remove the '-T' option from mergecap?Hi Hadriel, Please do not remove this option. It allows to fix wrong pcap files generated by other tools (see bug 11425 for example). Cheers, Pascal.
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- removing mergecap -T option Hadriel Kaplan (Aug 14)
- Re: removing mergecap -T option Pascal Quantin (Aug 14)
- Re: removing mergecap -T option Hadriel Kaplan (Aug 14)
- Re: removing mergecap -T option Pascal Quantin (Aug 14)
- Re: removing mergecap -T option Hadriel Kaplan (Aug 14)
- Re: removing mergecap -T option Guy Harris (Aug 14)
- Re: removing mergecap -T option Pascal Quantin (Aug 14)