Author Tony Grue
Recipients AndrewZiem, Tony Grue, amaury.forgeotdarc
Date 2016-06-02.01:55:50
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1464832550.85.0.385334345502.issue6443@psf.upfronthosting.co.za>
In-reply-to
Content
I see this was closed,though it sounds like it's because a work around is found, not because the code now handles multiple null values in a row.

Looking at code from the github mirror (pasted below); it looks like the issue is that the countStrings function breaks out of iterating in the for loop, even if it hasn't reached the end of the buffer.

File PC/winreg.c:

static int
countStrings(wchar_t *data, int len)
{
    int strings;
    wchar_t *P;
    wchar_t *Q = data + len;

    for (P = data, strings = 0; P < Q && *P != '\0'; P++, strings++)
        for (; P < Q && *P != '\0'; P++)
            ;
    return strings;
}
History
Date User Action Args
2016-06-02 01:55:50Tony Gruesetrecipients: + Tony Grue, amaury.forgeotdarc, AndrewZiem
2016-06-02 01:55:50Tony Gruesetmessageid: <1464832550.85.0.385334345502.issue6443@psf.upfronthosting.co.za>
2016-06-02 01:55:50Tony Gruelinkissue6443 messages
2016-06-02 01:55:50Tony Gruecreate