[PATCH] Enforce strict overflow checking

Stefan Metzmacher metze at samba.org
Fri Mar 23 15:41:34 UTC 2018


Am 23.03.2018 um 16:32 schrieb Andreas Schneider via samba-technical:
> On Friday, 23 March 2018 15:56:24 CET Stefan Metzmacher wrote:
>> Am 23.03.2018 um 15:36 schrieb Andreas Schneider via samba-technical:
>>> On Wednesday, 21 March 2018 18:26:04 CET Jeremy Allison wrote:
>>>> Andreas, I think you're going to have to work this
>>>> through on sn-devel first if it's going to go
>>>> through.
>>>
>>> OK, the attached patchset builds on sn-devel (gcc is more picky there) and
>>> a private autobuild is currently running and more than 60% of the tests
>>> already passed.
>>>
>>> The ldb_qsort was tricky so I wrote a test for it.
>>>
>>> I've set '-Wstrict-overflow=1' for bison generated getdate.c which uses
>>> C89
>>> and our old nmbd code.
>>
>> We can't add a hardcoded "-Wstrict-overflow=1" to every build.
>>
>> We may have to just add -Wstrict-overflow=2 to PICKY_FLAGS,
>> so that this isn't added with 'allow_warnings=True'.
>>
>> That would be a huge advantage already and makes sure we don't regress
>> in most of our code.
> 
> Then we could do it like HAVE_WNO_UNUSED_FUNCTION and turn the warning off. 
> See attached.

Thanks! That's much better.

metze


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20180323/d409c3e5/signature.sig>


More information about the samba-technical mailing list