Andres Freund <> writes:
> On 2014-06-03 23:19:37 +0900, Fujii Masao wrote:
>> -                    if (sscanf(copybuf + 124, "%11o", &current_len_left) != 
>> 1)
>> +                    if (sscanf(copybuf + 124, "%11lo", &current_len_left) 
>> != 1)

> That's probably not going to work on 32bit platforms or windows where
> you might need to use ll instead of l as a prefix. Also notice that
> apparently (c.f. 9d7ded0f4277f5c0063eca8e871a34e2355a8371) sscanf can't
> reliably be used for 64bit input :(. That pretty much sucks...

There's a far bigger problem there, which is if we're afraid that
current_len_left might exceed 4GB then what is it exactly that guarantees
it'll fit in an 11-digit field?

                        regards, tom lane

Sent via pgsql-hackers mailing list (
To make changes to your subscription:

Reply via email to