Re: Patch for snprintf problem (bug #1000650) - Mailing list pgsql-odbc

From Hiroshi Inoue
Subject Re: Patch for snprintf problem (bug #1000650)
Date
Msg-id 4488B6EA.9050107@tpf.co.jp
Whole thread Raw
In response to Patch for snprintf problem (bug #1000650)  (Ludek Finstrle <luf@pzkagis.cz>)
Responses Re: Patch for snprintf problem (bug #1000650)  (Ludek Finstrle <luf@pzkagis.cz>)
List pgsql-odbc
Ludek Finstrle wrote:
> Hello,
>
>   I found the problem in snprintf on linux (maybe another unix) boxes
> in info.c (CVS HEAD). The problematic part is something like:
>
> buf = "text";
> snprintf(buf,size,"%s append",buf);
>
> buf = "text append" on Windows (MS VC compiler)
> buf = " append" on linux (gcc compiler)
>
> I solve it this way (main idea):
> snprintf(buf + strlen(buf), " append");

Hmm bad news.
If so, it may be better to use the sequence like the following for example.

    char    *query_ptr;
    size_t    bufsize_res;
    int    slen;

    /* Initialize */
    query_ptr = columns_query;
    bufsize_res = sizeof(columns_query);

    if (..)
    {
        if ((slen = snprintf(query_ptr, bufsize_res, .., )) <= 0)
        {
            .. error_handling ..
        }
        query_ptr += slen;
        bufsize_res -= slen;
    }

    ...

regards,
Hiroshi Inoue

pgsql-odbc by date:

Previous
From: "Mr. DBA"
Date:
Subject: windows ODBC driver issues
Next
From: Hiroshi Inoue
Date:
Subject: Re: CVS and open cursor