Bug #8577
opendcerpc: bind PDUs with 0 pfc_flags don't match without any_frag
Description
Reported by @alexey -
-PCAP to be provided.
The issue was misrepresented. Conversation on this issue provides clarification.
SB Updated by Shivani Bhardwaj 1 day ago
- Related to Bug #8457: dcerpc.iface keyword matches any interface if PFC_FIRST_FRAG is missing in the BIND request added
PA Updated by Philippe Antoine about 20 hours ago
Pcap is already merged in SV with previous ticket
PA Updated by Philippe Antoine about 20 hours ago
If we agree this is the desired behavior, we need mostly to update the doc
AM Updated by Alexey Monastyrskiy about 15 hours ago
any_frag does not match on PDU with pfc_flags set to 0
I think the issue being discussed is that a lack of `any_frag` makes the signature not match on single-fragment BIND PDUs with pfc_flags set to 0. The PCAP is the same as in #8457. The test is the same as provided in #8457 too. The one that got merged in SV was changed to only match on the session with "normal" BIND pfc_flags, but originally I made it to also match on zeroed-out pfc_flags. (The PCAP contains two sessions.)
SB Updated by Shivani Bhardwaj about 7 hours ago ยท Edited
- Subject changed from dcerpc: any_frag does not match on PDU with pfc_flags set to 0 to dcerpc: bind PDUs with 0 pfc_flags don't match without any_frag
- Status changed from New to Rejected
Thank you, alexey ! won't show me packets with flags set to 0 but I had to use an explicit
I apologize. I was using an incorrect Wireshark filter on the PCAP you provided and misunderstood you based on that.
I thought @dcerpc.cn_flags.first_fragdcerpc.cn_flags.first_frag > 0
Based on Philippe's suggestion, I am rejecting this ticket and just keeping the doc update one. Thanks a lot for patiently and actively responding!
SB Updated by Shivani Bhardwaj about 7 hours ago
- Description updated (diff)
- Assignee deleted (
OISF Dev) - Target version deleted (
9.0.0-beta1)
SB Updated by Shivani Bhardwaj about 7 hours ago
- Description updated (diff)
PA Updated by Philippe Antoine about 3 hours ago
- Status changed from Rejected to New
Based on Philippe's suggestion, I am rejecting this ticket
That is not what I meant
I meant the code change is less than one line, but the doc change should be much bigger...