Home page logo
/

wireshark logo Wireshark mailing list archives

Re: [Wireshark-commits] master 0d2dd00: Fix -Wabsolute-value in ICMP dissector (found by Clang 3.5)
From: "Maynard, Chris" <Christopher.Maynard () GTECH COM>
Date: Sat, 15 Mar 2014 10:32:21 -0400

I don't think this is the correct fix.  The intent was to get the absolute value of the *difference* between 2 values, 
which could be negative, hence "abs(a - b)".

-----Original Message-----
From: wireshark-commits-bounces () wireshark org [mailto:wireshark-commits-bounces () wireshark org] On Behalf Of 
Wireshark code review
Sent: Saturday, March 15, 2014 7:38 AM
To: wireshark-commits () wireshark org
Subject: [Wireshark-commits] master 0d2dd00: Fix -Wabsolute-value in ICMP dissector (found by Clang 3.5)

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0d2dd005647d173959fdd1794ff04840eb5e8cd4
Submitter: Anders Broman (a.broman58 () gmail com)
Changed: branch: master
Repository: wireshark

Commits:

0d2dd00 by Alexis La Goutte (alexis.lagoutte () gmail com):

    Fix -Wabsolute-value in ICMP dissector (found by Clang 3.5)
    
    packet-icmp.c:1245:7: error: taking the absolute value of unsigned type 'unsigned int' has no effect 
[-Werror,-Wabsolute-value]
    packet-icmp.c:1245:7: note: remove the call to 'abs' since unsigned values cannot be negative
    packet-icmp.c:1245:30: error: taking the absolute value of unsigned type 'unsigned int' has no effect 
[-Werror,-Wabsolute-value]
    packet-icmp.c:1245:30: note: remove the call to 'abs' since unsigned values cannot be negative
    packet-icmp.c:1254:6: error: taking the absolute value of unsigned type 'unsigned int' has no effect 
[-Werror,-Wabsolute-value]
    packet-icmp.c:1254:6: note: remove the call to 'abs' since unsigned values cannot be negative
    packet-icmp.c:1254:29: error: taking the absolute value of unsigned type 'unsigned int' has no effect 
[-Werror,-Wabsolute-value]
    packet-icmp.c:1254:29: note: remove the call to 'abs' since unsigned values cannot be negative
    packet-icmp.c:1623:7: error: taking the absolute value of unsigned type 'guint32' (aka 'unsigned int') has no 
effect [-Werror,-Wabsolute-value]
    packet-icmp.c:1623:7: note: remove the call to 'abs' since unsigned values cannot be negative
    packet-icmp.c:1629:7: error: taking the absolute value of unsigned type 'guint32' (aka 'unsigned int') has no 
effect [-Werror,-Wabsolute-value]
    packet-icmp.c:1629:7: note: remove the call to 'abs' since unsigned values cannot be negative
    
    Change-Id: I6b344d01b8239fb93aedf95d954ef1243ba45a6b
    Reviewed-on: https://code.wireshark.org/review/673
    Reviewed-by: Anders Broman <a.broman58 () gmail com>
    

Actions performed:

    from  3094062   Use constants instead of vwr->STATS_LEN in some cases.
    adds  0d2dd00   Fix -Wabsolute-value in ICMP dissector (found by Clang 3.5)


Summary of changes:
 epan/dissectors/packet-icmp.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-) 
___________________________________________________________________________
Sent via:    Wireshark-commits mailing list <wireshark-commits () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
             mailto:wireshark-commits-request () wireshark org?subject=unsubscribe
CONFIDENTIALITY NOTICE: The information contained in this email message is intended only for use of the intended 
recipient. If the reader of this message is not the intended recipient, you are hereby notified that any dissemination, 
distribution or copying of this communication is strictly prohibited. If you have received this communication in error, 
please immediately delete it from your system and notify the sender by replying to this email.  Thank you.
___________________________________________________________________________
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

  By Date           By Thread  

Current thread:
  • Re: [Wireshark-commits] master 0d2dd00: Fix -Wabsolute-value in ICMP dissector (found by Clang 3.5) Maynard, Chris (Mar 15)
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]