Thread (8 messages) flat view 8 messages, 3 authors, 2016-06-15

Re: [PATCH v5 4/5] merge/pull Check for untrusted good GPG signatures

From: Thomas Rast <hidden>
Date: 2016-06-15 22:56:35

"Sebastian Götte" [off-list ref] wrote:
expecting success: 
test_must_fail git merge --ff-only --verify-signatures side-untrusted
2>mergeerror &&
       test_i18ngrep "has a good, untrusted GPG signature" mergeerror

==1430== Conditional jump or move depends on uninitialised value(s)
==1430==    at 0x4C26B5C: strchrnul (mc_replace_strmem.c:711)
==1430==    by 0x47B90B: check_commit_signature (commit.c:1057)
==1430==    by 0x444212: cmd_merge (merge.c:1245)
==1430==    by 0x4050E6: handle_internal_command (git.c:281)
==1430==    by 0x40530C: main (git.c:489)

Though I also can't see the problem. strchrnul gets passed a char* that
is just fine.
Usually that means that the string *contents* are uninitialized, usually because you scanned past the end of the string...
-- 
http://code.google.com/p/k9mail
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help