Re: fix for palloc() of user-supplied length

From: Serguei Mokhov <mokhov(at)cs(dot)concordia(dot)ca>
To: Neil Conway <neilc(at)samurai(dot)com>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: fix for palloc() of user-supplied length
Date: 2002-08-28 04:12:26
Message-ID: 1030507946.3d6c4daa4fc22@mailhost.cs.concordia.ca
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

I wrote:

> > I've attached a revised patch that includes the additional check Tom
> > suggested (len < 1). Unless anyone else steps forward, I'm inclined to
>
> + if (len < 1 || len > 8192)
> + {
> + elog(LOG, "Password packet length too long: %d", len);
> ^^^^^^^^
> Shouldn't it be changed to 'too long || too long' then? ;)

A typo: [too short or too short] :)

--
Serguei A. Mokhov, <mailto: mokhov @ cs.concordia.ca>
Computer Science, Concordia University

-------------------------------------------------
This mail sent through IMP: http://horde.org/imp/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2002-08-28 04:29:16 Re: [SQL] LIMIT 1 FOR UPDATE or FOR UPDATE LIMIT 1?
Previous Message Serguei Mokhov 2002-08-28 04:07:22 Re: fix for palloc() of user-supplied length

Browse pgsql-patches by date

  From Date Subject
Next Message Neil Conway 2002-08-28 04:33:26 Re: fix for palloc() of user-supplied length
Previous Message Serguei Mokhov 2002-08-28 04:07:22 Re: fix for palloc() of user-supplied length