From: | Ludek Finstrle <luf(at)pzkagis(dot)cz> |
---|---|
To: | Hiroshi Inoue <inoue(at)tpf(dot)co(dot)jp> |
Cc: | pgsql-odbc(at)postgresql(dot)org |
Subject: | Re: Patch for snprintf problem (bug #1000650) 5-th try |
Date: | 2006-06-14 07:50:54 |
Message-ID: | 20060614075054.GA10109@soptik.pzkagis.cz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-odbc |
Tue, Jun 13, 2006 at 11:12:58PM +0900, Hiroshi Inoue wrote:
> Ludek Finstrle wrote:
> >
> > I make patch againist CVS after yours huge commit. What's your opinion?
> >
>
> Is the second parameter of snprintf_add needed ?
> Aren't the parameter values always strlen(the first parameter) ?
You're right. I think more about it and "add" means add to the end
so I changed the patch as you pointed.
> Is snprintf_len needed instead of snprintf ?
> Though the current code ignores snprintf errors, it's simply
> my negligence..
I'm voting for keeping safer snprintf_len. But I can change it if
you wish.
New patch attached.
Regards,
Luf
Attachment | Content-Type | Size |
---|---|---|
psqlodbc_snprintf_5try.diff | text/plain | 12.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ludek Finstrle | 2006-06-14 08:09:34 | Re: CVS and open cursor |
Previous Message | noreply | 2006-06-14 07:49:16 | [ psqlodbc-Bugs-1000414 ] file based dns does not connect |