Re: BUG #16188: The usage of NULL pointer in rege_dfa.c - Mailing list pgsql-bugs

From Tom Lane
Subject Re: BUG #16188: The usage of NULL pointer in rege_dfa.c
Date
Msg-id 739.1578283741@sss.pgh.pa.us
Whole thread Raw
In response to BUG #16188: The usage of NULL pointer in rege_dfa.c  (PG Bug reporting form <noreply@postgresql.org>)
List pgsql-bugs
PG Bug reporting form <noreply@postgresql.org> writes:
> We checked the code in file “rege_dfa.c” and there is one error occurring in
> line 481. This error is caused by the usage of pointer with NULL value. The
> code in this line is “d->work = &d->statesarea[nss * wordsper];” We checked
> the code two lines before the error occurred line and the code is
> “d->statesarea = (unsigned *) MALLOC((nss + WORK) * wordsper
> *sizeof(unsigned));”. We found the code hasn’t checked weather the process
> “MALLOC” is success or not which directly define the value of the
> “statesarea” field of pointer “d”.

I don't see any bug there.  If the malloc for statesarea fails, the
code will exit a few lines further down, without ever having dereferenced
d->work.  So whether d->work is a valid pointer or not is irrelevant.

            regards, tom lane



pgsql-bugs by date:

Previous
From: PG Bug reporting form
Date:
Subject: BUG #16191: Errors of division by zero occur in several files.
Next
From: Tom Lane
Date:
Subject: Re: BUG #16189: The usage of NULL pointer in indexpath.c