Avoid returning undefined bytes in chkpass_in().
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 14 Feb 2015 17:20:56 +0000 (12:20 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 14 Feb 2015 17:20:56 +0000 (12:20 -0500)
commit80986e85aafd64165efc7f45b330f2f213713032
tree76a81a800f440da5e8635ca7b7f2bac59a1fea3c
parent33e879c4e9f485405fc19b663392d8635714d020
Avoid returning undefined bytes in chkpass_in().

We can't really fix the problem that the result is defined to depend on
random(), so it is still going to fail the "unstable input conversion"
test in parse_type.c.  However, we can at least satify valgrind.  (It
looks like this code used to be valgrind-clean, actually, until somebody
did a careless s/strncpy/strlcpy/g on it.)

In passing, let's just make real sure that chkpass_out doesn't overrun
its output buffer.

No need for backpatch, I think, since this is just to satisfy debugging
tools.

Asif Naeem
contrib/chkpass/chkpass.c