zorba-coders team mailing list archive

[Matthias Brantner <matthias.brantner@xxxxxxxxxx>]

Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug-http_parse-fragment into lp:zorba.

Commit message:
fixed crash if parse-fragment is used on an http stream

Requested reviews:
  Matthias Brantner (matthias-brantner)
  Nicolae Brinza (nbrinza)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-http_parse-fragment/+merge/157991

fixed crash if parse-fragment is used on an http stream
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-http_parse-fragment/+merge/157991
Your team Zorba Coders is subscribed to branch lp:zorba.

=== modified file 'modules/com/zorba-xquery/www/modules/http-client.xq.src/http_response_parser.cpp'
--- modules/com/zorba-xquery/www/modules/http-client.xq.src/http_response_parser.cpp	2013-02-07 17:24:36 +0000
+++ modules/com/zorba-xquery/www/modules/http-client.xq.src/http_response_parser.cpp	2013-04-10 05:23:23 +0000
@@ -286,6 +286,9 @@
 
   static void streamReleaser(std::istream* aStream)
   {
+    if (!aStream)
+      return;
+
     // This istream contains our curl stream buffer, so we have to delete it too
     std::streambuf *const sbuf = aStream->rdbuf();
     if ( transcode::streambuf *tbuf =