Fix a memory access and a double slash bug.
authorWerner Koch <wk@gnupg.org>
Thu, 8 May 2014 18:35:57 +0000 (20:35 +0200)
committerWerner Koch <wk@gnupg.org>
Thu, 8 May 2014 18:35:57 +0000 (20:35 +0200)
commitde4a1ea684e1591975feb801e7651309e1ee2c49
tree0ff9ef71c8fa36809d10e00bd04c5b5a4ff4252e
parentd5fb92cdaed21eea2f1a921e4f11df72635a8462
Fix a memory access and a double slash bug.

* src/engine-spawn.c (engspawn_start): Allocate space for list
terminator.
* src/posix-util.c (walk_path): Fix trailing slash detection.
--

Kudos to Valgrind for pointing out these two problems.

The first is a plain allocation bug in a code pattern I have written
thousands of times - this time it went wrong.  The allocation is not
user controlled thus not directly exploitable.

The second is missed to do what it intended to do.  Found due to the
access of malloced but not initialized memory.  Not using calloc
again proved to be helpful to detect logical error.
src/engine-spawn.c
src/posix-util.c