Re: [PATCH] pgcrypto: Test for NULL before dereferencing pointer

From: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
To: Marti Raudsepp <marti(at)juffo(dot)org>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCH] pgcrypto: Test for NULL before dereferencing pointer
Date: 2010-10-20 19:34:06
Message-ID: 4CBF442E.3060301@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 20.10.2010 18:44, Marti Raudsepp wrote:
> Hi pgsql-hackers,
>
> Currently contrib/pgcrypto/pgp-pubenc.c contains code like:
>
> uint8 algo = pk->algo;
> if (pk == NULL)
> ...
>
> However, if pk was NULL, then the if() condition would never be
> reached because the pk->algo dereference would segfault.
>
> This patch moves the dereference to below the condition which was the
> intended behavior.

Thanks, applied. Did coccicheck find anything else interesting?

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2010-10-20 19:39:09 Re: PostgreSQL and HugePage
Previous Message Alvaro Herrera 2010-10-20 19:33:12 Re: Review: Fix snapshot taking inconsistencies