]> rtime.felk.cvut.cz Git - coffee/buildroot.git/commit
mcookie: correct wrong memset argument
authorPeter Korsgaard <peter@korsgaard.com>
Wed, 10 Jan 2018 22:03:03 +0000 (23:03 +0100)
committerPeter Korsgaard <peter@korsgaard.com>
Fri, 12 Jan 2018 18:57:19 +0000 (19:57 +0100)
commit40f4191f2a1246b792ffc0c02b6c9bd2d62649f2
tree494433f1c5c2f53910ac0b9eb666e358f02a77d4
parent6f1c11f79a64387c1f1749550804f8aae0cfa7a7
mcookie: correct wrong memset argument

Fixes #10216

Building mcookie generates a warning about possible wrong arguments to
memset:

mcookie.c:207:26: warning: argument to ‘sizeof’ in ‘memset’ call is the same expression
  as the destination; did you mean to dereference it? [-Wsizeof-pointer-memaccess]
     memset(ctx, 0, sizeof(ctx)); /* In case it's sensitive */

ctx is a pointer to a structure, so the code should use the size of the
structure and not the size of the pointer when it tries to clear the
structure, similar to how it got fixed upstream back in 2009:

https://git.kernel.org/pub/scm/utils/util-linux/util-linux.git/commit/lib/md5.c?id=6596057175c6ed342dc20e85eae8a42eb29b629f

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
package/x11r7/mcookie/mcookie.c