~ubuntu-branches/debian/experimental/ettercap/experimental

Description: <short summary of the patch>

 TODO: Put a short summary on the line above and replace this paragraph

 with a longer explanation of this change. Complete the meta-information

 with other relevant fields (see below for details). To make it easier, the

 information below has been extracted from the changelog. Adjust it or drop

 it.

 .

 ettercap (1:0.7.5.1-3) unstable; urgency=low

 .

   * remove bogus but harmlessly dead code from patch for CVE-2013-0722

Author: Barak A. Pearlmutter <bap@debian.org>

 

---

The information above should follow the Patch Tagging Guidelines, please

checkout http://dep.debian.net/deps/dep3/ to learn about the format. Here

are templates for supplementary fields that you might want to add:

 

Origin: <vendor|upstream|other>, <url of original patch>

Bug: <url in upstream bugtracker>

Bug-Debian: http://bugs.debian.org/<bugnumber>

Bug-Ubuntu: https://launchpad.net/bugs/<bugnumber>

Forwarded: <no|not-needed|url proving that it has been forwarded>

Reviewed-By: <name and email of someone who approved the patch>

Last-Update: <YYYY-MM-DD>

 

--- ettercap-0.7.5.1.orig/include/ec_inet.h

+++ ettercap-0.7.5.1/include/ec_inet.h

@@ -24,24 +24,19 @@

    #endif

 #endif

 

-enum {

-   NS_IN6ADDRSZ            = 16,

-   NS_INT16SZ              = 2,

-

-   ETH_ADDR_LEN            = 6,

-   TR_ADDR_LEN             = 6,

-   FDDI_ADDR_LEN           = 6,

-   MEDIA_ADDR_LEN          = 6,

+#define        ETH_ADDR_LEN 6

+#define        TR_ADDR_LEN 6

+#define        FDDI_ADDR_LEN 6

+#define        MEDIA_ADDR_LEN 6

    

-   IP_ADDR_LEN             = 4,

-   IP6_ADDR_LEN            = 16,

-   MAX_IP_ADDR_LEN         = IP6_ADDR_LEN,

+#define        IP_ADDR_LEN 4

+#define        IP6_ADDR_LEN 16

+#define        MAX_IP_ADDR_LEN IP6_ADDR_LEN

 

-   ETH_ASCII_ADDR_LEN      = sizeof("ff:ff:ff:ff:ff:ff")+1,

-   IP_ASCII_ADDR_LEN       = sizeof("255.255.255.255")+1,

-   IP6_ASCII_ADDR_LEN      = sizeof("ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255")+1,

-   MAX_ASCII_ADDR_LEN      = IP6_ASCII_ADDR_LEN,                  

-};

+#define        ETH_ASCII_ADDR_LEN 19 // sizeof("ff:ff:ff:ff:ff:ff")+1

+#define        IP_ASCII_ADDR_LEN 17 // sizeof("255.255.255.255")+1

+#define        IP6_ASCII_ADDR_LEN 47 // sizeof("ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255")+1

+#define        MAX_ASCII_ADDR_LEN IP6_ASCII_ADDR_LEN

 

 /*

  * Some predefined addresses here

--- ettercap-0.7.5.1.orig/include/ec.h

+++ ettercap-0.7.5.1/include/ec.h

@@ -81,6 +81,11 @@

 

 #define SAFE_FREE(x) do{ if(x) { free(x); x = NULL; } }while(0)

 

+

+/* convert to string */

+#define EC_STRINGIFY(in) #in

+#define EC_TOSTRING(in) EC_STRINGIFY(in)

+

 #ifdef OS_LINUX

 #define __init       __attribute__((constructor(101)))

 #define __init_last  __attribute__((constructor(200))

--- ettercap-0.7.5.1.orig/src/ec_scan.c

+++ ettercap-0.7.5.1/src/ec_scan.c

@@ -630,7 +630,7 @@ int scan_load_hosts(char *filename)

    for (nhosts = 0; !feof(hf); nhosts++) {

       int proto;

 

-      if (fscanf(hf, "%s %s %s\n", ip, mac, name) != 3 ||

+      if (fscanf(hf, "%"EC_TOSTRING(MAX_ASCII_ADDR_LEN)"s %"EC_TOSTRING(ETH_ASCII_ADDR_LEN)"s %"EC_TOSTRING(MAX_HOSTNAME_LEN)"s\n", ip, mac, name) != 3 ||

          *ip == '#' || *mac == '#' || *name == '#')

          continue;

 

--- ettercap-0.7.5.1.orig/src/dissectors/ec_mongodb.c

+++ ettercap-0.7.5.1/src/dissectors/ec_mongodb.c

@@ -99,7 +99,7 @@ FUNC_DECODER(dissector_mongodb)

               if (session_get(&s, ident, DISSECT_IDENT_LEN) == ESUCCESS) {

                       conn_status = (struct mongodb_status *) s->data;

                       if (PACKET->DATA.len < 16)

-                              return;

+                              return NULL;

                       unsigned char *res = memmem(ptr, PACKET->DATA.len, "fails", 5);

                       unsigned char *gres = memmem(ptr, PACKET->DATA.len, "readOnly", 8);

                       if (conn_status->status == WAIT_RESULT && res) {

@@ -118,7 +118,7 @@ FUNC_DECODER(dissector_mongodb)

       if (session_get(&s, ident, DISSECT_IDENT_LEN) == ESUCCESS) {

          conn_status = (struct mongodb_status *) s->data;

          if (PACKET->DATA.len < 16)

-                 return;

+                 return NULL;

 

          unsigned char *noncep  = memmem(ptr, PACKET->DATA.len, "nonce", 5);

          unsigned char *keyp  = memmem(ptr, PACKET->DATA.len, "key\x00", 4);