git.samba.org / idra / samba.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: cabd9e7) | patch
sasl_secret_t ends in a char [1] size. This means the extra character is implicit...
authorJeremy Allison <jra@samba.org>
Tue, 3 May 2011 20:53:30 +0000 (13:53 -0700)
committerJeremy Allison <jra@samba.org>
Wed, 4 May 2011 19:12:14 +0000 (12:12 -0700)
commit5c53d63348882b17f16bed0cc41f1489dcd6cf66
tree3c9f5c24da6a42c5f2f66cfc57b4ceb5cd4f182ftree
parentcabd9e70e2f3ce3f617975a4d005d79121313b5fcommit | diff
sasl_secret_t ends in a char [1] size. This means the extra character is implicit in the safe_strcpy. When changing to strlcpy ensure we allocate an extra char for it. This fixes a bug where secret->len+1 used with safe_strcpy could actually write into secret->len+2.
source4/auth/gensec/cyrus_sasl.c diff | blob | history
Work In Progress