Michael Paquier <michael.paqu...@gmail.com> wrote: > Coverity is pointing out that _int_split.c has unnecessary checks > for deferenced pointers in 5 places.
> Attached is a patch to adjust those things. Pushed. Thanks! > Also, as far as I understood from this code, no elements > manipulated are NULL, perhaps this is worth an assertion? I'm not clear where you were thinking of, but anyway that seemed like a separate patch if we're going to do it, so I went ahead with pushing the issued Coverity flagged. The arguments to the function don't need such a check because the function is exposed to SQL with the STRICT option (but you probably already knew that). While reviewing the safety of this patch the only place that I ran across that I felt maybe deserved an assertion was that n >= 0 near the top of copy_intArrayType(), but that seems marginal. -- Kevin Grittner EDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers