[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Gnash-dev] Re: test_http failure and memory errors in rtmp branch

From: strk
Subject: [Gnash-dev] Re: test_http failure and memory errors in rtmp branch
Date: Fri, 12 Dec 2008 18:35:39 +0100

On Fri, Dec 12, 2008 at 06:24:12PM +0100, strk wrote:
> I'm getting a failure in testsuite/libnet.all/test_http
> from the rtmp branch. But the result is unsable to I checked
> with valgrind and found a few conditional jumps based
> on uninitialized values, right before the failing test.

It turns out it was missing initialization of the _version
member of HTTP class. It is a structure, I suggest equipping
it with a default constructor to avoid this in the future.

The attached patch tries to be as unobtrusive as possible
by explicitly initializing the struct members in the HTTP
constructors (2 of them). Gives consistent successes
with test_http.

Initializing in the http_version_t struct ctor itself would be
more robust.


Attachment: http._version.init.patch
Description: Text document

reply via email to

[Prev in Thread] Current Thread [Next in Thread]