#23 URLStream patch

closed
None
5
2002-02-25
2002-02-25
Anonymous
No

Default TCP port is set to 0 in CVS version (in
function URLStream::sendHttpHeader()). Fetching from
URL of type <prot>://<address>/<path> hangs forever in
TCPStream::connect method (where it calls
std::connect()) or fails with reason 1
(errUnreachable). To verify bug try compiling
demo/urlfetch and use `urlfetch http://www.slackware.com\`

This patch is against CVS. Sorry guys, but this is my
first patch ever, so I'm not sure about the submission
guidelines.

I generated this patch using diff -rub olddir/ newdir/
> url.patch. The syntax seems a bit different from
other patches here. Could anyone please give me a
suggestion as to how exactly can one generate those
patches? Thanks.

Discussion

  • Nobody/Anonymous

    Set default port in sendHttpHeader() to 80

     
  • Frediano Ziglio

    Frediano Ziglio - 2002-02-25
    • assigned_to: nobody --> freddy77
    • status: open --> closed-fixed
     
  • Frediano Ziglio

    Frediano Ziglio - 2002-02-25

    Logged In: YES
    user_id=75766

    applied
    ps: your patch was perfect

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks