看板FB_security
标 题Re: OpenSSL static analysis, was: De Raadt + FBSD + OpenSSH + hole?
发信站NCTU CS FreeBSD Server (Sat Apr 26 15:40:18 2014)
转信站ptt!csnews.cs.nctu!news.cednctu!FreeBSD.cs.nctu!.POSTED!freebsd.org!ow
On 25 April 2014 22:21, Ronald F. Guilmette <
[email protected]> wrote:
>
> In message <CAG5KPzw_cOfFLX_kn=5DWAX+z+9VeXuzo3Q8YekDJG37tDQ_wQ@mail.gmail.com>
> Ben Laurie <[email protected]> wrote:
>
>>But that would then hide the error condition of it being not set to a
>>new value after initialisation.
>
> The (modified/quieted) code example under discussion is as follows:
>
> variable = value0; /* initialization */
> if (condition) variable = value1;
> if (!condition) variable = value2;
> use (variable);
>
> Please note that variable *is* *always* set to some value (either value1
> or value2) after initialization. The "error condition" that you seem
> concerned about having explicitly flagged does not in fact exist in the
> example code snippet under discussion.
Of course it doesn't, that's the whole point.
>>One better answer would be to have a way to annotate that after the
>>two conditionals you assert that |variable| is initialised. Then a
>>future, smarter static analyzer can attempt to prove you wrong.
>
> Sir, is there any sense in which the final line of the above example
> _does not_ already and standing all by itself constitute an implicit
> assertion that "variable" has been initialized by that point in the
> code?
My point is that if you want to silence the warning, the right way to
do it is by saying "you are wrong, here is my claimed truth" rather
than by changing the code to avoid the shortcoming in analysis. The
reason being that future analysis can decide to ignore or disprove
your assertion, but cannot ignore the suppression you suggest.
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-security
To unsubscribe, send any mail to "
[email protected]"