Conversation

Also, signed overflow being undefined rather than defined as wrapping means that more secure implementations where it traps are permitted. Passing -fsanitize=signed-integer-overflow -fsanitize-trap=signed-integer-overflow is standards compliant and used for hardening in AOSP.
3
5
That code can be fixed and the fixes are clear cut bug fixes. High quality C code is tested with ASan, TSan, UBSan, etc. and many of these issues are already being caught and fixed over time. Portable and safe C code needs to avoid relying on undefined behavior like this.
2
5
C isn't defined as that language, and you're not in a position where you get to define the language. In the real world, C is deployed with various safety features taking advantage of many things being undefined and reducing portability / compatibility with those wouldn't be good.
2
5
This Tweet was deleted by the Tweet author. Learn more
K&R C explicitly permits arbitrary implementations of overflow including an implementation that traps. It doesn't mandate two's complement or any specific set of possible implementations. ANSI C refers to standard C containing all the explicit wording about undefined behavior.
1
1
K&R gives the compiler and hardware the ability to do whatever they want in this case. They mention that current compilers do not try to detect overflow. It's explicitly permitted to catch overflow. Per K&R, even unsigned integer overflow is allowed to be handled differently.
1
So if you want to talk about K&R C, -fsanitize=signed-integer-overflow,unsigned-integer-overflow -fsanitize-trap=signed-integer-overflow,unsigned-integer-overflow conforms to how it's defined, unlike ANSI / ISO C where only signed overflow checking is compliant with the standard.
1