Re: Add error-checking to timestamp_recv

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Add error-checking to timestamp_recv
Date: 2004-05-20 12:42:32
Message-ID: 20040520124232.GS11196@ns.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

* Bruce Momjian (pgman(at)candle(dot)pha(dot)pa(dot)us) wrote:
> Would you show an example of the invalid value this is trying to avoid?

Well, the way I discovered the problem was by sending a timestamp in
double format when the server was expecting one in int64 format. This
is when using the binary data method for timestamps. I'll generate a
small example program/schema later today and post it to the list.

Stephen

> ---------------------------------------------------------------------------
>
> Stephen Frost wrote:
> > Greetings,
> >
> > The attached patch adds some error-checking to the timestamp_recv
> > function so that it's not possible to put an invalid timestamp into a
> > timestamp column (hopefully). The check is done in basically the
> > exact same way the out-of-bounds check in timestamp2tm is done.
> > There's probably an easier/cleaner way but this should work or at
> > least generate discussion and a better patch. :)
> >
> > Thanks,
> >
> > Stephen
>
> [ Attachment, skipping... ]
>
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 6: Have you searched our list archives?
> >
> > http://archives.postgresql.org
>
> --
> Bruce Momjian | http://candle.pha.pa.us
> pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
> + If your life is a hard drive, | 13 Roberts Road
> + Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2004-05-20 14:32:18 Re: Add error-checking to timestamp_recv
Previous Message Andrew Dunstan 2004-05-20 11:34:03 Re: pgkill for windows