Merge lp:~paul-lucas/zorba/bug-950456 into lp:zorba

Proposed by Paul J. Lucas on 2012-03-09
Status: Merged
Approved by: Matthias Brantner on 2012-03-09
Approved revision: 10711
Merged at revision: 10711
Proposed branch: lp:~paul-lucas/zorba/bug-950456
Merge into: lp:zorba
Diff against target: 24 lines (+3/-0)
1 file modified
modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp (+3/-0)
To merge this branch: bzr merge lp:~paul-lucas/zorba/bug-950456
Reviewer Review Type Date Requested Status
Matthias Brantner 2012-03-09 Approve on 2012-03-09
Paul J. Lucas Approve on 2012-03-09
Review via email: mp+96693@code.launchpad.net

Commit Message

Added delete theSerStream;

Description of the Change

Added delete theSerStream;

To post a comment you must log in.
Paul J. Lucas (paul-lucas) :
review: Approve
review: Approve
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job bug-950456-2012-03-09T01-14-13.39Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp'
2--- modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp 2011-06-14 14:21:49 +0000
3+++ modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp 2012-03-09 01:04:20 +0000
4@@ -39,6 +39,7 @@
5 theSerStream(NULL),
6 thePost(NULL),
7 theLast(NULL),
8+ theLastSerializerOptions(NULL),
9 theIsHeadRequest(false)
10 {
11 theHeaderLists.push_back(NULL);
12@@ -57,6 +58,7 @@
13 if (thePost != NULL) {
14 curl_formfree(thePost);
15 }
16+ delete theSerStream;
17 }
18
19 void HttpRequestHandler::begin()
20@@ -288,3 +290,4 @@
21
22 } // namespace http_client
23 } // namespace zorba
24+/* vim:set et sw=2 ts=2: */

Subscribers

People subscribed via source and target branches