Wireshark mailing list archives
Re: Expected ABI breakage with 1.6.8
From: Balint Reczey <balint.reczey () ericsson com>
Date: Tue, 22 May 2012 00:47:33 +0200
On 05/21/2012 09:43 PM, Gerald Combs wrote:
On 5/21/12 12:39 PM, Guy Harris wrote:On May 21, 2012, at 12:02 PM, Balint Reczey wrote:Please do not release 1.6.8 in its current state or bump library version to 2.0.0 for libwireshark. Using 1.1.8 is misleading as 1.6.8 breaks the ABI: http://rbalint.cs.bme.hu/ws-ABI-1.6.7-1.6.8/libwireshark/abi_compat_report.htmlSummary: The data_rate field in the struct ieee_802_11_phdr structure grew from 8 bits to 16 bits. This was a fix for bug 7134: r42123 | alagoutte | 2012-04-18 00:22:12 -0700 (Wed, 18 Apr 2012) | 14 lines Changed paths: M /trunk/wiretap/wtap.h From Colus Tang via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7134 Unable to display the correct IEEE802.11 MCS data rates due to header definition The problem is due to the ieee_802_11_phdr.data_rate is defined as guint8, since this variable is counting number of 0.5Mbps units, any datarates which is higher than 255Mbps would get wrapped up. In the above example, only the lower 8bit value will be put into the ieee_802_11_phdr which is 0x04 and result in the incorrect 2Mbps display. There are 802.11n WLAN product is capable to transmit @450Mbps, we should fix this data_rate from guint8 to guint16. #BACKPORT This does *NOT* affect radiotap captures or any other captures where the 802.11 information is presented as a header at the beginning of the packet data. It affects OmniPeek captures as they currently use ieee_802_11_phdr (which prevents some items in OmniPeek's radio data from being presented). I'd say "un-backport and require anybody who wants full 802.11n support in OmniPeek captures to upgrade to 1.8".Done in r42757. Balint, can you check the updated ABI on your end?
Hi Gerald, Thanks, the ABI is now compatible with 1.6.7. Cheers, Balint PS: The logs: comparing headers ... comparing libraries ... creating compatibility report ... result: COMPATIBLE (total problems: 0, warnings: 0) see detailed report: compat_reports/libwiretap/1.0.7_to_1.0.8/abi_compat_report.html preparation, please wait ... comparing headers ... comparing libraries ... creating compatibility report ... result: COMPATIBLE (total problems: 0, warnings: 0) see detailed report: compat_reports/libwsutil/1.0.0_to_1.0.0/abi_compat_report.html preparation, please wait ... Using GCC 4.4.5 (i486-linux-gnu) checking header(s) 1.0.0 ... Using C99 compatibility mode checking header(s) 1.0.0 ... Using C99 compatibility mode comparing headers ... comparing libraries ... creating compatibility report ... result: COMPATIBLE (total problems: 0, warnings: 0) see detailed report: compat_reports/libwsutil/1.0.0_to_1.0.0/abi_compat_report.html preparation, please wait ... comparing headers ... comparing libraries ... creating compatibility report ... result: COMPATIBLE (total problems: 0, warnings: 0) see detailed report: compat_reports/libwireshark/1.1.7_to_1.1.8/abi_compat_report.html preparation, please wait ... Using GCC 4.4.5 (i486-linux-gnu) checking header(s) 1.1.7 ... Using C99 compatibility mode ERROR: some errors occurred when compiling headers ERROR: see log for details: /.../projects/wireshark.git/logs/libwireshark/1.1.7/log.txt checking header(s) 1.1.8 ... Using C99 compatibility mode ERROR: some errors occurred when compiling headers ERROR: see log for details: /.../projects/wireshark.git/logs/libwireshark/1.1.8/log.txt comparing headers ... comparing libraries ... creating compatibility report ... result: COMPATIBLE (total problems: 0, warnings: 1) see detailed report: compat_reports/libwireshark/1.1.7_to_1.1.8/abi_compat_report.html ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Expected ABI breakage with 1.6.8 Balint Reczey (May 21)
- Re: Expected ABI breakage with 1.6.8 Guy Harris (May 21)
- Re: Expected ABI breakage with 1.6.8 Gerald Combs (May 21)
- Re: Expected ABI breakage with 1.6.8 Balint Reczey (May 21)
- Re: Expected ABI breakage with 1.6.8 Gerald Combs (May 21)
- Re: Expected ABI breakage with 1.6.8 Guy Harris (May 21)