[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-12 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 Pascal Quantin changed: What|Removed |Added Priority|High|Low CC|

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-12 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 Pascal Quantin changed: What|Removed |Added See Also||https://bugs.sequans.com/sh

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-12 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 Pascal Quantin changed: What|Removed |Added See Also|https://bugs.sequans.com/sh |https://bugs.wireshark.org/

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-12 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #2 from hhw --- (In reply to hhw from comment #0) > Created attachment 15442 [details] > illegally sms reassembly due to have different sm-RP-DA > > Build Information: > Version 2.0.2 (v2.0.2-0-ga16e22e from master-2.0) > -

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-13 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 shafaghja...@gmail.com changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCO

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-22 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 hhw changed: What|Removed |Added Priority|Low |Medium -- You are receiving this mail beca

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-23 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 Pascal Quantin changed: What|Removed |Added Severity|Major |Enhancement -- You are receivin

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-23 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #3 from Gerrit Code Review --- Change 21306 had a related patch set uploaded by Pascal Quantin: GSM SMS: optionally use GSM-MAP info to perform reassembly (off by default) https://code.wireshark.org/review/21306 -- You are r

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #4 from hhw --- (In reply to Gerrit Code Review from comment #3) > Change 21306 had a related patch set uploaded by Pascal Quantin: > GSM SMS: optionally use GSM-MAP info to perform reassembly (off by default) > > https://code

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #5 from Pascal Quantin --- It means that you will have a GSM SMS dissector preference to toggle to activate the feature. It is off by default as I found some previous captures where the SM-RP-OA/DA field is not always present,

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #6 from hhw --- (In reply to Pascal Quantin from comment #5) > It means that you will have a GSM SMS dissector preference to toggle to > activate the feature. > It is off by default as I found some previous captures where the S

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #7 from Pascal Quantin --- If you took a precompiled binary, this is normal as the patch was not merged yet. If you built yourself, then in GSM SMS dissector options you must activate the new setting asking to take into account

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #8 from hhw --- (In reply to Pascal Quantin from comment #7) > If you took a precompiled binary, this is normal as the patch was not merged > yet. > If you built yourself, then in GSM SMS dissector options you must activate > t

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #9 from hhw --- in GSM_SMS over gsm_map-tcap We have 2 scenario: Scenario 1: Each part of sms submit/deliver in a Separate dialog(Transaction Capabilities Application Part), in this scenario each part has sm_RP_DA field that

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #10 from Pascal Quantin --- (In reply to hhw from comment #9) > in GSM_SMS over gsm_map-tcap We have 2 scenario: > > Scenario 1: > Each part of sms submit/deliver in a Separate dialog(Transaction > Capabilities Application Pa

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #11 from hhw --- More details are in attachments:scenario1.PNG, scenario2.PNG -- You are receiving this mail because: You are watching all bug changes.__

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #12 from hhw --- Created attachment 15500 --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=15500&action=edit scenario1 of sms reassembly -- You are receiving this mail because: You are watching all bug changes.

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #13 from hhw --- Created attachment 15501 --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=15501&action=edit scenario2 of sms reassembly -- You are receiving this mail because: You are watching all bug changes.

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #14 from hhw --- (In reply to hhw from comment #11) > More details are in attachments:scenario1.PNG, scenario2.PNG in some cases because of delay in network, segments are not submited /deliverd Respectively -- You are recei

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #15 from Gerrit Code Review --- Change 21320 had a related patch set uploaded by Pascal Quantin: GSM SMS: add more lower layer info for reassembly and switch preference to on by default https://code.wireshark.org/review/21320

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-24 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 hhw changed: What|Removed |Added Attachment #15501|0 |1 is obsolete|

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-26 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #17 from Gerrit Code Review --- Change 21354 had a related patch set uploaded by Pascal Quantin: GSM SMS: Use SIP To / From tapped addresses for reassembly https://code.wireshark.org/review/21354 -- You are receiving this ma

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-26 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #18 from hhw --- (In reply to Gerrit Code Review from comment #17) > Change 21354 had a related patch set uploaded by Pascal Quantin: > GSM SMS: Use SIP To / From tapped addresses for reassembly > > https://code.wireshark.org/

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-26 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #19 from hhw --- (In reply to Pascal Quantin from comment #10) > (In reply to hhw from comment #9) > > in GSM_SMS over gsm_map-tcap We have 2 scenario: > > > > Scenario 1: > > Each part of sms submit/deliver in a Separate dia

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-26 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #20 from Pascal Quantin --- As explained previously, this is not straightforward. So it will be done once someone feels motivated enough to do it. -- You are receiving this mail because: You are watching all bug changes._

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-28 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #21 from Gerrit Code Review --- Change 21397 had a related patch set uploaded by Pascal Quantin: GSM MAP: use TCP OTID to retrieve SM-RP-OA and SM-RP-DA fields https://code.wireshark.org/review/21397 -- You are receiving thi

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-28 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #22 from Gerrit Code Review --- Change 21397 merged by Anders Broman: GSM MAP: use TCAP OTID to retrieve SM-RP-OA and SM-RP-DA fields when required https://code.wireshark.org/review/21397 -- You are receiving this mail becau

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 Pascal Quantin changed: What|Removed |Added Resolution|--- |FIXED Status|CONFIRM

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #24 from hhw --- (In reply to Pascal Quantin from comment #23) > For now I let S1AP / RANAP out of scope. thank you very much. but I have a question. In new attachment(SMS_IMSI2) the sm_RP_DA of the 3 parts are same and we are

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #25 from hhw --- Created attachment 15513 --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=15513&action=edit SMS_Reassembly -- You are receiving this mail because: You are watching all bug changes._

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #26 from hhw --- hello. i have a question. In multipart SMS, if all parts are existed then wireshark is reassembeld them. i want to reassemble a multipart SMS even though one part is not existed ? for example(attachment:sms_pa

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #28 from Pascal Quantin --- (In reply to hhw from comment #24) > (In reply to Pascal Quantin from comment #23) > > For now I let S1AP / RANAP out of scope. > > thank you very much. but I have a question. > In new attachment(SM

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #27 from hhw --- Created attachment 15514 --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=15514&action=edit sms_part -- You are receiving this mail because: You are watching all bug changes.___

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #29 from Pascal Quantin --- (In reply to hhw from comment #26) > hello. i have a question. > > In multipart SMS, if all parts are existed then wireshark is reassembeld > them. i want to reassemble a multipart SMS even though o

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #30 from hhw --- (In reply to Pascal Quantin from comment #28) > (In reply to hhw from comment #24) > > (In reply to Pascal Quantin from comment #23) > > > For now I let S1AP / RANAP out of scope. > > > > thank you very much.

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-29 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #31 from hhw --- (In reply to Pascal Quantin from comment #28) > (In reply to hhw from comment #24) > > (In reply to Pascal Quantin from comment #23) > > > For now I let S1AP / RANAP out of scope. > > > > thank you very much.

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #32 from Pascal Quantin --- If you crafted again a capture where you purposely removed the packet giving the sm-rp-oa/da real value, then the behavior seen is expected. The otid is used to retrieve the previously seen sm-rp-oa/

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #33 from hhw --- I should say that all of the bugs I said is in a real traffic and the last problem also is in a real traffic! -- You are receiving this mail because: You are watching all bug changes._

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #34 from Pascal Quantin --- So you have a network sending SMS (with test content) to multiple UEs with the same message identifier each time and lost packets. This is unlucky. For your last use case, no GSM MAP identity is not

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #35 from Pascal Quantin --- We could tighten even more the checks but in case of filtered packets you might end with non reassembled packets while they should, or the opposite. OTID could presumably be reused later on. The code

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #36 from shafaghja...@gmail.com --- what an interesting topic! I read all the comments and I think for the last bug (comment 31) if we have this situation (in a real traffic or hand made data) wireshark should not reassemble th

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #37 from Pascal Quantin --- (In reply to shafaghjahad from comment #36) > what an interesting topic! > I read all the comments and I think for the last bug (comment 31) if we have > this situation (in a real traffic or hand ma

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #38 from shafaghja...@gmail.com --- (In reply to Pascal Quantin from comment #28) > (In reply to hhw from comment #24) > > (In reply to Pascal Quantin from comment #23) > > > For now I let S1AP / RANAP out of scope. > > > > tha

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #39 from Pascal Quantin --- (In reply to shafaghjahad from comment #38) > (In reply to Pascal Quantin from comment #28) > > (In reply to hhw from comment #24) > > > (In reply to Pascal Quantin from comment #23) > > > > For now

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #40 from shafaghja...@gmail.com --- (In reply to Pascal Quantin from comment #39) > (In reply to shafaghjahad from comment #38) > > (In reply to Pascal Quantin from comment #28) > > > (In reply to hhw from comment #24) > > > > (

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #41 from Gerrit Code Review --- Change 21425 had a related patch set uploaded by Pascal Quantin: GSM SMS: use TCAP OTID as last resort for reassembly when GSM MAP does not give info https://code.wireshark.org/review/21425 --

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #42 from Pascal Quantin --- (In reply to Gerrit Code Review from comment #41) > Change 21425 had a related patch set uploaded by Pascal Quantin: > GSM SMS: use TCAP OTID as last resort for reassembly when GSM MAP does not > giv

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #43 from hhw --- (In reply to Pascal Quantin from comment #42) > (In reply to Gerrit Code Review from comment #41) > > Change 21425 had a related patch set uploaded by Pascal Quantin: > > GSM SMS: use TCAP OTID as last resort f

[Wireshark-bugs] [Bug 13592] GSM SMS dissector does not take into account lower layer info for reassembly

2017-04-30 Thread bugzilla-daemon
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13592 --- Comment #44 from Gerrit Code Review --- Change 21425 merged by Pascal Quantin: GSM SMS: use TCAP OTID as last resort for reassembly when GSM MAP does not give info https://code.wireshark.org/review/21425 -- You are receiving this ma