You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Rob Hartill <ro...@imdb.com> on 1996/06/17 17:40:27 UTC

Re: WWW Form Bug Report: "httpd: caught SIGBUS, dumping core, caused by lack of HTTP/ in GET, fix found" on HPUX

Thanks for the bug report and a bigger thank you for the patch.

We should be able to get this fixed for 1.1b4 due real soon now.

regards,
rob

>Submitter: ccount@mit.edu
>Operating system: HPUX, version: 
>Version of Apache Used: 1.1b3
>Extra Modules used: 
>URL exhibiting problem: 
>
>Symptoms:
>--
>[Mon Jun 17 11:12:34 1996] httpd: caught SIGBUS, dumping core
>This happens when no HTTP version is provided
>with a GET, e.g.
>telnet tantalum.mit.edu 80
>GET foo
>
>produces a sigbus.
>I traced it to http_protocol.c, it's passing sscanf a constant
>string.  This is bad under gcc and HPUX.
>
>A fix is:
>
>*** http_protocol.c~    Mon Jun 17 11:08:00 1996
>--- http_protocol.c     Mon Jun 17 11:17:09 1996
>***************
>*** 325,331 ****
>      parse_uri (r, uri);
>      
>      r->assbackwards = (ll[0] == '\0');
>!     r->protocol = ll[0] ? pstrdup (r->pool, ll) : "HTTP/0.9";
>      sscanf(r->protocol, "HTTP/%d.%d", &major, &minor);
>      r->proto_num = 1000*major + minor;
>  
>--- 325,332 ----
>      parse_uri (r, uri);
>      
>      r->assbackwards = (ll[0] == '\0');
>! /*    r->protocol = ll[0] ? pstrdup (r->pool, ll) : "HTTP/0.9";*/
>!     r->protocol = pstrdup (r->pool, ll[0] ?  ll : "HTTP/0.9");
>      sscanf(r->protocol, "HTTP/%d.%d", &major, &minor);
>      r->proto_num = 1000*major + minor;
>  
>
>--
>
>Backtrace:
>--
>
>--

-- 
Rob Hartill (robh@imdb.com)
The Internet Movie Database (IMDb)  http://www.imdb.com/
           ...more movie info than you can poke a stick at.

Re: WWW Form Bug Report: "httpd: caught SIGBUS, dumping core, caused by lack of HTTP/ in GET, fix found" on HPUX

Posted by Alexei Kosut <ak...@organic.com>.
On Mon, 17 Jun 1996, Rob Hartill wrote:

> >produces a sigbus.
> >I traced it to http_protocol.c, it's passing sscanf a constant
> >string.  This is bad under gcc and HPUX.

Not on the gcc and HP-UX I use... but I don't see anything wrong with the
patch. I'll toss it onto the 1.1b4 pile.

-- Alexei Kosut <ak...@organic.com> 
   http://www.nueva.pvt.k12.ca.us/~akosut/