155
67%

Ran 30 Mar 2017 07:27PM UTC

Jobs 1

Files 67

Run time 3s

Badge

Embed ▾

pending completion

Build # 155

Build Type

push

travis-ci

Committed by

darnir

Commit Message

Fix memory leak in test-metalink

This memory leak was particularly interesting. Valgrind did *NOT* report
the leak at all while GCC's Memory Sanitizer did, leading me to believe
that this is a bug in the memory sanitizer. However, the memory leak
does indeed exist and Valgrind misses it.

The leak happens because the url->body is always statically allocated.
_If_ we replace the {{port}} in the body, the new string is dynamically
allocated and stored in the struct. However, if the user stores a
dynamically allocated memory in the struct, then we replace it during
wget_test_start_server leaking that memory.

* tests/test-metalink.c: Explicitly deallocate dynamically allocated
memory

Run Details

5904 of 8955 relevant lines covered (65.93%)

3337.82 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	155.1	30 Mar 2017 07:27PM UTC	0	65.93	Travis Job 155.1

Source Files on build 155

Detailed source file information is not available for this build.

darnir / wget2 / 155
67%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 155

darnir / wget2 / 155 67%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 155

darnir / wget2 / 155
67%

README BADGES
x