Wireshark-commits: [Wireshark-commits] rev 32772: /trunk/epan/dissectors/ /trunk/epan/dissectors/:

Date: Wed, 12 May 2010 13:24:47 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=32772

User: morriss
Date: 2010/05/12 06:24 AM

Log:
 From Dirk via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4758 :
 
 The HTTP dissector uses strtoll() to convert the Content-Length string into a
 64bit variable. But that string can contain a number larger (or less) than
 64bit, which lets the strtoll() return INT_MAX (or INT_MIN). strtoll() then
 indicates this with errno==ERANGE.
 
 The attachted patch checks if errno is set this way and then treats that HTTP
 Content-Length as unspecified, since we don't know the real size.
 
 I haven't checked other occurences of strtoll() in the HTTP dissector if they
 could benefit from the errno check, or if other dissectors could use it.

Directory: /trunk/epan/dissectors/
  Changes    Path             Action
  +11 -7     packet-http.c    Modified