On 14 December 2011 14:54, Henri Verbeet <[email protected]> wrote:
> On 14 December 2011 14:44, Michael Curran <[email protected]> wrote:
>> Just a quick check, since Debian had a big kerfuffle when someone went
>> and initialized variables in a crypto module, but is there any chance
>> that those variables were left uninitialized on purpose?
>>
> Except for some TRACEs, I don't think these are actually read.
And actually, that probably makes the patch wrong too. If those
functions end up being called by an application you'd still be tracing
uninitialized data, but Coverity just wouldn't notice.


Reply via email to