Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 23 Dec 2012 17:16:49 +0100
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: scan-build results, part 1

On 23 Dec, 2012, at 17:12 , Dhiru Kholia <dhiru.kholia@...il.com> wrote:

> On Sun, Dec 23, 2012 at 8:27 PM, magnum <john.magnum@...hmail.com> wrote:
>> On 23 Dec, 2012, at 15:39 , magnum <john.magnum@...hmail.com> wrote:
>>> On 23 Dec, 2012, at 15:09 , Dhiru Kholia <dhiru.kholia@...il.com> wrote:
>>>> For complete set of results, see
>>>> http://dl.dropbox.com/u/1522424/scan-build-2012-12-23-16.tar.bz2
>>> 
>>> I just fixed the options.c issue. I'm trying to understand the unicode.c one...
>> 
>> I really can't see why it complains about "Branch condition evaluates to a garbage value" in unicode.c. It seems to think the tmp16u can remain undefined but I fail to see how that could ever happen.
> 
> scan-build also gives its "reasoning" in a graphical way. That might
> help. I am taking a look.

Another weird complaint is for MSCHAPv2. The pos pointer is set to non-null in line 429. How could it ever be a null dereference in line 433?

magnum

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.