diff options
| author | Mark Walters <markwalters1009@gmail.com> | 2012-08-08 22:23:33 +0100 |
|---|---|---|
| committer | David Bremner <bremner@debian.org> | 2012-08-12 21:25:01 +0200 |
| commit | 94c3b40d41f1ad98719d411ef28b69075fda0579 (patch) | |
| tree | 626dd70f0fc46947595312aae8c62e1320d6bf68 /test/missing-headers | |
| parent | 36e640852b240ad10daed626391e1b812df89200 (diff) | |
sprinters: bugfix when NULL passed for a string.
The string function in a sprinter may be called with a NULL string
pointer (eg if a header is absent). This causes a segfault. We fix
this by checking for a null pointer in the string functions and update
the sprinter documentation.
At the moment some output when format=text is done directly rather than
via an sprinter: in that case a null pointer is passed to printf or
similar and a "(null)" appears in the output. That behaviour is not
changed in this patch.
Diffstat (limited to 'test/missing-headers')
| -rwxr-xr-x | test/missing-headers | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/test/missing-headers b/test/missing-headers index e79f9227..f14b8784 100755 --- a/test/missing-headers +++ b/test/missing-headers @@ -29,7 +29,6 @@ thread:XXX 2001-01-05 [1/1] (null); (inbox unread) thread:XXX 1970-01-01 [1/1] Notmuch Test Suite; (inbox unread)" test_begin_subtest "Search: json" -test_subtest_known_broken output=$(notmuch search --format=json '*' | notmuch_search_sanitize) test_expect_equal_json "$output" ' [ @@ -93,7 +92,6 @@ Body message}" test_begin_subtest "Show: json" -test_subtest_known_broken output=$(notmuch show --format=json '*' | notmuch_json_show_sanitize) test_expect_equal_json "$output" ' [ |