Wireshark-commits: [Wireshark-commits] master eb56b71: RTS_FLAG_NONE is a flags field value, but it

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 3 Nov 2016 01:58:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=eb56b7172d33605cbb969afd91beb302f89013ed
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

eb56b71 by Guy Harris (guy@xxxxxxxxxxxx):

    RTS_FLAG_NONE is a flags field value, but it's not a flag bit.
    
    It's the *absence* of flag bits, so you can't test whether it's set;
    don't have a field for it.  This gets rid of a dissector assertion.
    
    Add some more URLs for information about DCE RPC and MS-RPC.
    
    Change-Id: I53e8f11692c66e16ae2ec7fd4ba8eb90b0673da4
    Reviewed-on: https://code.wireshark.org/review/18648
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  46160ff   proto: Decode negative values in a bitmask field
    adds  eb56b71   RTS_FLAG_NONE is a flags field value, but it's not a flag bit.


Summary of changes:
 epan/dissectors/packet-dcerpc.c |   12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)