From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 8432F386CE61; Tue, 14 Jun 2022 05:09:48 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8432F386CE61 From: "siddhesh at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug middle-end/101836] __builtin_object_size(P->M, 1) where M is an array and the last member of a struct fails Date: Tue, 14 Jun 2022 05:09:48 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: middle-end X-Bugzilla-Version: 12.0 X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: siddhesh at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: qinzhao at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 14 Jun 2022 05:09:48 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D101836 --- Comment #22 from Siddhesh Poyarekar --- (In reply to Kees Cook from comment #21) > How about "-fnot-flex-arrays=3DN" to mean "trailing arrays with N or more > elements will NOT be treated like a flex array"? >=20 > Then code with sockaddr can use "-fnot-flex-arrays=3D15", code with "[1]" > arrays can use "-fnot-flex-arrays=3D2", code with only "[0]" arrays can u= se > "-fnot-flex-arrays=3D1", and "-fstrict-flex-arrays" can be an alias for > "-fnot-flex-arrays=3D0", which Linux would use. An arbitrary N will only make it abuse-friendly and potentially mask bugs. = IMO if we choose to make multiple levels here it should only be -fstrict-flex-arrays=3D{1,2} where 1 (the default) only allows "[]" and 2 a= llows "[0]", disabling all other size values. For anything else, -fno-strict-flex-arrays. My opinion on the default is not strong FWIW.=