]> granicus.if.org Git - clang/commit
Sema: Warn on sizeof on binary ops on decayed arrays.
authorBenjamin Kramer <benny.kra@googlemail.com>
Fri, 29 Mar 2013 21:43:21 +0000 (21:43 +0000)
committerBenjamin Kramer <benny.kra@googlemail.com>
Fri, 29 Mar 2013 21:43:21 +0000 (21:43 +0000)
commit52b2e7085f09bf7834b41f6e807aff5ac97bd3a5
treec9d811aec773665c8096a45f211851ed03c62441
parent9b97adfb770c3b55c1a45049d53b624bbc6f62e1
Sema: Warn on sizeof on binary ops on decayed arrays.

The array will decay into a pointer, creating an unexpected result.
sizeof(array + int) is an easy to make typo for sizeof(array) + int.

This was motivated by a NetBSD security bug, used sizeof(key - r) instead of
sizeof(key) - r, reducing entropy in a random number generator.
http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/kern/subr_cprng.c.diff?r1=1.14&r2=1.15&only_with_tag=MAIN&f=h

Differential Revision: http://llvm-reviews.chandlerc.com/D571

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@178371 91177308-0d34-0410-b5e6-96231b3b80d8
include/clang/Basic/DiagnosticGroups.td
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExpr.cpp
test/Sema/expr-comma-c99.c
test/Sema/expr-comma.c
test/Sema/warn-sizeof-array-decay.c [new file with mode: 0644]