On Thu, Oct 27, 2011 at 08:38, Eric Pouech <eric.pou...@orange.fr> wrote: > why do we check for ptr being null or not, when we deref ptr one line above?
>> if (*ptr == '\n') ptr++; >> - WCMD_output_asis_len(message, (ptr) ? ptr - message : >> strlenW(message), handle); >> + WCMD_output_asis_len(message, (ptr) ? (DWORD)(ptr - message) : >> strlenW(message), handle); >> if (ptr) { >> numChars = 0; >> if (++line_count >= max_height - 1) { It's dereferenced but incremented afterwards, so might be null after that line