[pkgsrc/pkgsrc-2008Q4]: pkgsrc/net/wireshark pullup ticket #2697 - requested ...

[rtr <rtr%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/7a81e271a8fd
branches:  pkgsrc-2008Q4
changeset: 552332:7a81e271a8fd
user:      rtr <rtr%pkgsrc.org@localhost>
date:      Mon Feb 16 05:21:51 2009 +0000

description:
pullup ticket #2697 - requested by tron
wireshark: update package for security fix

revisions pulled up:
pkgsrc/net/wireshark/Makefile           1.29
pkgsrc/net/wireshark/PLIST              1.12
pkgsrc/net/wireshark/distinfo           1.19
pkgsrc/net/wireshark/patches/patch-ad   r0

   Module Name:    pkgsrc
   Committed By:   tron
   Date:           Fri Feb 13 09:41:11 UTC 2009

   Modified Files:
           pkgsrc/net/wireshark: Makefile PLIST distinfo
   Removed Files:
           pkgsrc/net/wireshark/patches: patch-ad

   Log Message:
   Update "wireshark" package to version 1.0.6.

   Changes between 1.0.5 and 1.0.6.:
   - The following vulnerabilities have been fixed:
     * On non-Windows systems, Wireshark could crash if the HOME environment
       variable contained sprintf-style string formatting characters.
     * Wireshark could crash while reading a malformed NetScreen snoop file.
     * Wireshark could crash while reading a Tektronix K12 text capture file.
   - The following bugs have been fixed:
     * Crash when loading capture file and Preferences: NO Info column
     * Some Lua scripts may lead to corruption via out of bounds stack
     * Build with GLib 1.2 fails with error: 'G_MININT32' undeclared
     * Wrong decoding IMSI with GSM MAP protocol
     * Segmentation fault for "Follow TCP stream" (Bug 3119)
     * SMPP optional parameter 'network_error_code' incorrectly decoded
     * DHCPv6 dissector doesn't handle malformed FQDN
     * WCCP overrides CFLOW as decoded protocol (Bug 3175)
     * Improper decoding of MPLS echo reply IPv4 Interface and Label Stack Object
     * ANSI MAP fix for TRN digits/SMS and OTA subdissection (Bug 3214)
   - Updated Protocol Support
     * AFS, ATM, DHCPv6, DIS, E.212, RTP, UDP, USB, WCCP, WPS
   - New and Updated Capture File Support
     * NetScreen snoop

   Changes between version 1.0.4 and 1.0.5:
   - The following vulnerabilities have been fixed. See the security advisory
     for details and a workaround.
     * The SMTP dissector could consume excessive amounts of CPU and memory.
     * The WLCCP dissector could go into an infinte loop.
   - The following bugs have been fixed:
     * Missing CRLF during HTTP POST in the "packet details" window
     * Memory assertion in time_secs_to_str_buf() when compiled with GCC 4.2.3
     * Diameter dissector fails RFC 4005 compliance
     * LDP vendor private TLV type is not correctly shown
     * Wireshark on MacOS does not run when there are spaces in its path
     * Compilation broke when compiling without zlib
     * Memory leak: saved_repoid
     * Memory leak: follow_info
     * Memory leak: follow_info
     * Memory leak: tacplus_data
     * Memory leak: col_arrows
     * Memory leak: col_arrows
     * Incorrect address structure assigned for find_conversation() in WSP
     * Memory leak with unistim in voip_calls
     * Error parsing the BSSGP protocol
     * Assertion thrown in fvalue_get_uinteger when decoding TIPC
     * LUA script : Wireshark crashes after closing and opening again a window
       used by a listener.draw() function.
   - Updated Protocol Support
     * ANSI MAP, BSSGP, CIP, Diameter, ENIP, GIOP, H.263, H.264, HTTP, MPEG PES
     * PostgreSQL, PPI, PTP, Rsync, RTP, SMTP, SNMP, STANAG 5066, TACACS, TIPC
     * WLCCP, WSP

   The package update was provided by Matthias Drochner in private e-mail.

diffstat:

 net/wireshark/Makefile         |    5 +-
 net/wireshark/PLIST            |    4 +-
 net/wireshark/distinfo         |    9 +-
 net/wireshark/patches/patch-ad |  341 -----------------------------------------
 4 files changed, 9 insertions(+), 350 deletions(-)

diffs (truncated from 400 to 300 lines):

diff -r 66831b8d5f07 -r 7a81e271a8fd net/wireshark/Makefile
--- a/net/wireshark/Makefile    Sun Feb 15 22:24:06 2009 +0000
+++ b/net/wireshark/Makefile    Mon Feb 16 05:21:51 2009 +0000
@@ -1,7 +1,6 @@
-# $NetBSD: Makefile,v 1.28 2008/11/25 22:53:54 tron Exp $
+# $NetBSD: Makefile,v 1.28.2.1 2009/02/16 05:21:51 rtr Exp $
 
-DISTNAME=              wireshark-1.0.4
-PKGREVISION=           1
+DISTNAME=              wireshark-1.0.6
 CATEGORIES=            net
 MASTER_SITES=          http://www.wireshark.org/download/src/ \
                        ${MASTER_SITE_SOURCEFORGE:=wireshark/}
diff -r 66831b8d5f07 -r 7a81e271a8fd net/wireshark/PLIST
--- a/net/wireshark/PLIST       Sun Feb 15 22:24:06 2009 +0000
+++ b/net/wireshark/PLIST       Mon Feb 16 05:21:51 2009 +0000
@@ -1,4 +1,4 @@
-@comment $NetBSD: PLIST,v 1.11 2008/04/12 22:43:08 jlam Exp $
+@comment $NetBSD: PLIST,v 1.11.10.1 2009/02/16 05:21:51 rtr Exp $
 bin/capinfos
 bin/dftest
 bin/dumpcap
@@ -58,6 +58,7 @@
 share/wireshark/cfilters
 share/wireshark/colorfilters
 share/wireshark/dfilters
+share/wireshark/diameter/TGPPRx.xml
 share/wireshark/diameter/TGPPSh.xml
 share/wireshark/diameter/chargecontrol.xml
 share/wireshark/diameter/dictionary.dtd
@@ -104,6 +105,7 @@
 share/wireshark/radius/dictionary.alteon
 share/wireshark/radius/dictionary.altiga
 share/wireshark/radius/dictionary.aptis
+share/wireshark/radius/dictionary.aruba
 share/wireshark/radius/dictionary.ascend
 share/wireshark/radius/dictionary.bay
 share/wireshark/radius/dictionary.bintec
diff -r 66831b8d5f07 -r 7a81e271a8fd net/wireshark/distinfo
--- a/net/wireshark/distinfo    Sun Feb 15 22:24:06 2009 +0000
+++ b/net/wireshark/distinfo    Mon Feb 16 05:21:51 2009 +0000
@@ -1,9 +1,8 @@
-$NetBSD: distinfo,v 1.18 2008/11/25 22:53:54 tron Exp $
+$NetBSD: distinfo,v 1.18.2.1 2009/02/16 05:21:51 rtr Exp $
 
-SHA1 (wireshark-1.0.4.tar.bz2) = 8e75a6d909a1da803db77f6f86fdd5096e5bbac8
-RMD160 (wireshark-1.0.4.tar.bz2) = 741b6618ba34b55079f15d5725a1e9a22a4fc351
-Size (wireshark-1.0.4.tar.bz2) = 13126757 bytes
+SHA1 (wireshark-1.0.6.tar.bz2) = 1aa726e32905a2815d01a9f33df84a873b11c7ef
+RMD160 (wireshark-1.0.6.tar.bz2) = 5037c3c5f38c914233f064efb3dbe409896b3e0b
+Size (wireshark-1.0.6.tar.bz2) = 13161721 bytes
 SHA1 (patch-aa) = c155f38e66a553b14778dc73344b46f8614eb9b0
 SHA1 (patch-ab) = 5ae79916603f04c2d362c764d39f0c99728e716c
 SHA1 (patch-ac) = 4e985520ea4b118aea6fc001f256b5de96de7840
-SHA1 (patch-ad) = e19775622ed6facc4ab05ebd09059f78444f6c43
diff -r 66831b8d5f07 -r 7a81e271a8fd net/wireshark/patches/patch-ad
--- a/net/wireshark/patches/patch-ad    Sun Feb 15 22:24:06 2009 +0000
+++ /dev/null   Thu Jan 01 00:00:00 1970 +0000
@@ -1,341 +0,0 @@
-$NetBSD: patch-ad,v 1.1 2008/11/25 22:53:55 tron Exp $
-
---- epan/dissectors/packet-smtp.c.orig 2008-10-20 20:19:31.000000000 +0100
-+++ epan/dissectors/packet-smtp.c      2008-11-25 22:30:30.000000000 +0000
-@@ -101,10 +101,6 @@
-       "DATA fragments"
- };
- 
--/* Define media_type/Content type table */
--static dissector_table_t media_type_dissector_table;
--
--
- static  dissector_handle_t imf_handle = NULL;
- 
- /*
-@@ -179,6 +175,7 @@
-     gint                    length_remaining;
-     gboolean                eom_seen = FALSE;
-     gint                    next_offset;
-+    gint                    loffset;
-     gboolean                is_continuation_line;
-     int                     cmdlen;
-     fragment_data           *frag_msg = NULL;
-@@ -221,21 +218,6 @@
-      * longer than what's in the buffer, so the "tvb_get_ptr()" call
-      * won't throw an exception.
-      */
--    linelen = tvb_find_line_end(tvb, offset, -1, &next_offset,
--      smtp_desegment && pinfo->can_desegment);
--    if (linelen == -1) {
--      /*
--       * We didn't find a line ending, and we're doing desegmentation;
--       * tell the TCP dissector where the data for this message starts
--       * in the data it handed us, and tell it we need one more byte
--       * (we may need more, but we'll try again if what we get next
--       * isn't enough), and return.
--       */
--      pinfo->desegment_offset = offset;
--      pinfo->desegment_len = 1;
--      return;
--    }
--    line = tvb_get_ptr(tvb, offset, linelen);
- 
-     frame_data = p_get_proto_data(pinfo->fd, proto_smtp);
- 
-@@ -271,6 +253,42 @@
- 
-       }
- 
-+      if(request) {
-+      frame_data = se_alloc(sizeof(struct smtp_proto_data));
-+
-+      frame_data->conversation_id = conversation->index;
-+      frame_data->more_frags = TRUE;
-+
-+      p_add_proto_data(pinfo->fd, proto_smtp, frame_data);    
-+
-+      }
-+
-+    loffset = offset;
-+    while (tvb_offset_exists(tvb, loffset)) {
-+
-+    linelen = tvb_find_line_end(tvb, loffset, -1, &next_offset,
-+      smtp_desegment && pinfo->can_desegment);
-+    if (linelen == -1) {
-+
-+      if(offset == loffset) {
-+      /*
-+       * We didn't find a line ending, and we're doing desegmentation;
-+       * tell the TCP dissector where the data for this message starts
-+       * in the data it handed us, and tell it we need one more byte
-+       * (we may need more, but we'll try again if what we get next
-+       * isn't enough), and return.
-+       */
-+      pinfo->desegment_offset = loffset;
-+      pinfo->desegment_len = 1;
-+      return;
-+      }
-+      else {
-+      linelen = tvb_length_remaining(tvb, loffset);
-+      next_offset = loffset + linelen;
-+      }
-+    }
-+    line = tvb_get_ptr(tvb, loffset, linelen);
-+
-       /*
-        * Check whether or not this packet is an end of message packet
-        * We should look for CRLF.CRLF and they may be split.
-@@ -286,16 +304,16 @@
-        * .CRLF at the begining of the same packet.
-        */
- 
--      if ((request_val->crlf_seen && tvb_strneql(tvb, offset, ".\r\n", 3) == 0) ||
--          tvb_strneql(tvb, offset, "\r\n.\r\n", 5) == 0) {
-+      if ((request_val->crlf_seen && tvb_strneql(tvb, loffset, ".\r\n", 3) == 0) ||
-+          tvb_strneql(tvb, loffset, "\r\n.\r\n", 5) == 0) {
- 
-         eom_seen = TRUE;
- 
--      }
-+      } 
- 
--      length_remaining = tvb_length_remaining(tvb, offset);
--      if (length_remaining == tvb_reported_length_remaining(tvb, offset) &&
--          tvb_strneql(tvb, offset + length_remaining - 2, "\r\n", 2) == 0) {
-+      length_remaining = tvb_length_remaining(tvb, loffset);
-+      if (length_remaining == tvb_reported_length_remaining(tvb, loffset) &&
-+          tvb_strneql(tvb, loffset + length_remaining - 2, "\r\n", 2) == 0) {
- 
-         request_val->crlf_seen = TRUE;
- 
-@@ -314,11 +332,6 @@
- 
-       if (request) {
- 
--      frame_data = se_alloc(sizeof(struct smtp_proto_data));
--
--      frame_data->conversation_id = conversation->index;
--      frame_data->more_frags = TRUE;
--
-       if (request_val->reading_data) {
-         /*
-          * This is message data.
-@@ -333,6 +346,9 @@
-            */
-           frame_data->pdu_type = SMTP_PDU_EOM;
-           request_val->reading_data = FALSE;
-+          
-+          break;
-+          
-         } else {
-           /*
-            * Message data with no EOM.
-@@ -344,7 +360,7 @@
-              * We are handling a BDAT message.
-              * Check if we have reached end of the data chunk.
-              */
--            request_val->msg_read_len += tvb_length_remaining(tvb, offset);
-+            request_val->msg_read_len += tvb_length_remaining(tvb, loffset);
- 
-               if (request_val->msg_read_len == request_val->msg_tot_len) {
-               /* 
-@@ -360,6 +376,8 @@
-                  */
-                 frame_data->more_frags = FALSE;
-               }
-+              
-+              break; /* no need to go through the remaining lines */
-             }
-           }
-         }
-@@ -450,12 +468,15 @@
-           frame_data->pdu_type = request_val->data_seen ? SMTP_PDU_MESSAGE : SMTP_PDU_CMD;
- 
-         }
--
-       }
-+      }
- 
--      p_add_proto_data(pinfo->fd, proto_smtp, frame_data);
-+      /*
-+       * Step past this line.
-+       */
-+      loffset = next_offset;
- 
--      }
-+    }
-     }
- 
-     /*
-@@ -467,6 +488,7 @@
-       col_set_str(pinfo->cinfo, COL_PROTOCOL, "SMTP");
- 
-     if (check_col(pinfo->cinfo, COL_INFO)) {  /* Add the appropriate type here */
-+      col_clear(pinfo->cinfo, COL_INFO);
- 
-       /*
-        * If it is a request, we have to look things up, otherwise, just
-@@ -481,21 +503,38 @@
-       case SMTP_PDU_MESSAGE:
- 
-         length_remaining = tvb_length_remaining(tvb, offset);
--        col_set_str(pinfo->cinfo, COL_INFO, smtp_data_desegment ? "DATA fragment" : "Message Body");
-+        col_set_str(pinfo->cinfo, COL_INFO, smtp_data_desegment ? "C: DATA fragment" : "C: Message Body");
-         col_append_fstr(pinfo->cinfo, COL_INFO, ", %d byte%s", length_remaining,
-                         plurality (length_remaining, "", "s"));
-         break;
- 
-       case SMTP_PDU_EOM:
- 
--        col_add_fstr(pinfo->cinfo, COL_INFO, "EOM: %s",
--            format_text(line, linelen));
-+        col_set_str(pinfo->cinfo, COL_INFO, "C: .");
-+
-         break;
- 
-       case SMTP_PDU_CMD:
- 
--        col_add_fstr(pinfo->cinfo, COL_INFO, "Command: %s",
--            format_text(line, linelen));
-+        loffset = offset;
-+        while (tvb_offset_exists(tvb, loffset)) {
-+          /*
-+           * Find the end of the line.
-+           */
-+          linelen = tvb_find_line_end(tvb, loffset, -1, &next_offset, FALSE);
-+          line = tvb_get_ptr(tvb, loffset, linelen);
-+
-+          if(loffset == offset) 
-+            col_append_fstr(pinfo->cinfo, COL_INFO, "C: %s",
-+                         format_text(line, linelen));
-+          else {
-+            col_append_fstr(pinfo->cinfo, COL_INFO, " | %s",
-+                         format_text(line, linelen));
-+          }
-+
-+          loffset = next_offset;
-+
-+        }
-         break;
- 
-       }
-@@ -503,9 +542,24 @@
-       }
-       else {
- 
--      col_add_fstr(pinfo->cinfo, COL_INFO, "Response: %s",
--          format_text(line, linelen));
-+        loffset = offset;
-+        while (tvb_offset_exists(tvb, loffset)) {
-+          /*
-+           * Find the end of the line.
-+           */
-+          linelen = tvb_find_line_end(tvb, loffset, -1, &next_offset, FALSE);
-+          line = tvb_get_ptr(tvb, loffset, linelen);
-+
-+          if(loffset == offset) 
-+            col_append_fstr(pinfo->cinfo, COL_INFO, "S: %s",
-+                         format_text(line, linelen));
-+          else {
-+            col_append_fstr(pinfo->cinfo, COL_INFO, " | %s",