←back to thread

GCC 15.1

(gcc.gnu.org)
270 points jrepinc | 2 comments | | HN request time: 0.401s | source
Show context
Calavar ◴[] No.43792948[source]
> {0} initializer in C or C++ for unions no longer guarantees clearing of the whole union (except for static storage duration initialization), it just initializes the first union member to zero. If initialization of the whole union including padding bits is desirable, use {} (valid in C23 or C++) or use -fzero-init-padding-bits=unions option to restore old GCC behavior.

This is going to silently break so much existing code, especially union based type punning in C code. {0} used to guarantee full zeroing and {} did not, and step by step we've flipped the situation to the reverse. The only sensible thing, in terms of not breaking old code, would be to have both {0} and {} zero initialize the whole union.

I'm sure this change was discussed in depth on the mailing list, but it's absolutely mind boggling to me

replies(14): >>43793036 #>>43793080 #>>43793121 #>>43793150 #>>43793166 #>>43794045 #>>43794558 #>>43796460 #>>43798312 #>>43798826 #>>43800132 #>>43800234 #>>43800932 #>>43800975 #
mistrial9 ◴[] No.43793150[source]
using UNION was always considered sketchy IMHO. This is trivia for security exploiters?
replies(1): >>43793494 #
grandempire ◴[] No.43793494[source]
No. This is how sum types are implemented.

And from a runtime perspective it’s going to be a struct with perhaps more padding. You’ll need more details about your specific threat model to explain why that’s bad.

replies(1): >>43793585 #
mistrial9 ◴[] No.43793585[source]
a quick search says that std::variant is the modern replacement to implement your niche feature "sum types"
replies(3): >>43793601 #>>43794375 #>>43795744 #
1. jlouis ◴[] No.43794375[source]
Not a niche feature. Fundamental for any decent language with a type system.
replies(1): >>43799265 #
2. mistrial9 ◴[] No.43799265[source]
ok, but C99 and C++11 and others, all have ways to implement types. "Fundemental" as you say.. using UNION in C++ is not a good choice to implement types.. in old C99, you can use UNION that way but why? footguns all around.