Package: siege
Version: 2.61-1
Severity: normal
Tags: patch

Multiple bugs. First about not having understood how HTTP headers are
contructed ("=" instead of ":" as separator). Checking the validity of
values is, of course, beyond the thought anyway.

--- siege_orig/siege-2.61/src/http.c    2004-11-19 15:47:21.000000000
+0100
+++ siege/siege-2.61/src/http.c 2005-09-19 17:47:18.119287505 +0200
@@ -374,7 +374,11 @@
       else{
         h->auth.type.proxy = BASIC;
       }   
-      tmp = strchr( line, '=' );
+      tmp = strchr( line, ':' );
+      if (tmp == NULL) {
+        printf("I shat myself so hard..\n");
+        return NULL;
+      }
       tmp++;
       if( tmp[0] == '"' ){ tmp++; tmp[strlen(tmp)-1] = '\0'; }
       strncpy( h->auth.realm.proxy, tmp, strlen( tmp )); 


And in hash.c we're also back to not checking bloody values.

--- siege_orig/siege-2.61/src/hash.c    2003-07-09 22:22:38.000000000
+0200
+++ siege/siege-2.61/src/hash.c 2005-09-19 18:04:19.990104391 +0200
@@ -182,6 +182,7 @@
   int  x;
   NODE *node;
 
+  if (key == NULL) { return 1; }
   x = hash_genkey( this->size, key );
   for( node = this->table[x]; node != NULL; node = node->next ){
     if( !strcmp( node->key, key )){


cheers,
&rw


-- System Information:
Debian Release: 3.1
Architecture: i386 (i686)
Kernel: Linux 2.6.8-2-686
Locale: LANG=en_US, LC_CTYPE=en_US (charmap=ISO-8859-1)

Versions of packages siege depends on:
ii  libc6                         2.3.5-4    GNU C Library: Shared libraries an
ii  libssl0.9.7                   0.9.7e-3   SSL shared libraries

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to