pg.ddx.io pgsql-bugs@postgresql.org mailing list archive
help / color / mirror / Atom feedBUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
6+ messages / 3 participants
[nested] [flat]
* BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
@ 2019-11-29 19:40 PG Bug reporting form <noreply@postgresql.org>
0 siblings, 2 replies; 6+ messages in thread
From: PG Bug reporting form @ 2019-11-29 19:40 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org; +Cc: paul@intekon.com
The following bug has been logged on the website:
Bug reference: 16143
Logged by: Paul Spencer
Email address: paul@intekon.com
PostgreSQL version: 11.5
Operating system: Redhat and Debian
Description:
PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format
specifier %b is used. The returned month is one greater then the expected
month. If the expected month is “Dec”, the application may crash with a
segment fault. The format specifier %B has a similar issue.
** Investigation Notes
- The month is increased by one at line 143 in timestamp2tm() defined in
timestamp.c
https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/pgtypeslib/timestam...
- The month number is converted to the abbreviation at line 337 in
dttofmtasc_replace() defined in timestamp.c
https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/pgtypeslib/timestam...
- Month abbreviations are defined at line 499 in dt_common.c
https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/pgtypeslib/dt_commo...
***
* Use Case
***
pi@raspberrypi4:~/projects/postgres_month_date $ ./pg_month_date
PostgreSQL timestamp_fmt_asc()
PGTYPEStimestamp_to_asc returns 1999-01-08 04:06:06
Format string = %Y-%m-%d %H:%M:%S, Formated Date = 1999-01-08 04:06:06
Format string = %Y-%b-%d %H:%M:%S, Formated Date = 1999-Feb-08 04:06:06
pi@raspberrypi4:~/projects/postgres_month_date $
***
* Source code for the use case
***
/*
* pg_month_date.c
*/
#include <stdio.h>
#include <string.h>
#include "pgtypes_timestamp.h"
int main(int argc, char **argv)
{
char formatString[255] = "";
char stringBuffer[255] = "";
timestamp testTimestamp;
char * endPtr = NULL;
printf("%s\n","PostgreSQL timestamp_fmt_asc()");
testTimestamp = PGTYPEStimestamp_from_asc("1999-01-08 04:06:06",
&endPtr);
printf("PGTYPEStimestamp_to_asc returns %s\n",
PGTYPEStimestamp_to_asc(testTimestamp));
strncpy(formatString, "%Y-%m-%d %H:%M:%S", sizeof(formatString));
PGTYPEStimestamp_fmt_asc(&testTimestamp, stringBuffer,
sizeof(stringBuffer),formatString);
printf( "Format string = %s, Formated Date = %s\n", formatString ,
stringBuffer);
strncpy(formatString, "%Y-%b-%d %H:%M:%S", sizeof(formatString));
PGTYPEStimestamp_fmt_asc(&testTimestamp, stringBuffer,
sizeof(stringBuffer),formatString);
printf( "Format string = %s, Formated Date = %s\n", formatString ,
stringBuffer);
return 0;
}
Paul Spencer
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
@ 2019-11-29 20:38 Tomas Vondra <tomas.vondra@2ndquadrant.com>
parent: PG Bug reporting form <noreply@postgresql.org>
1 sibling, 1 reply; 6+ messages in thread
From: Tomas Vondra @ 2019-11-29 20:38 UTC (permalink / raw)
To: paul@intekon.com; pgsql-bugs@lists.postgresql.org
On Fri, Nov 29, 2019 at 07:40:37PM +0000, PG Bug reporting form wrote:
>The following bug has been logged on the website:
>
>Bug reference: 16143
>Logged by: Paul Spencer
>Email address: paul@intekon.com
>PostgreSQL version: 11.5
>Operating system: Redhat and Debian
>Description:
>
>PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format
>specifier %b is used. The returned month is one greater then the expected
>month. If the expected month is “Dec”, the application may crash with a
>segment fault. The format specifier %B has a similar issue.
>
>** Investigation Notes
>- The month is increased by one at line 143 in timestamp2tm() defined in
>timestamp.c
https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/pgtypeslib/timestam...
>- The month number is converted to the abbreviation at line 337 in
>dttofmtasc_replace() defined in timestamp.c
https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/pgtypeslib/timestam...
>- Month abbreviations are defined at line 499 in dt_common.c
>https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/pgtypeslib/dt_commo...
>
Yeah, seems like a simple off-by-one mistake. Our tm->tm_mon is 1-based,
but dttofmtasc_replace uses it directly to access elements of arrays
with month names. Hence the "next" month is returned, and crash for the
last month (access out of bounds).
The attached patch should fix this, I believe - both for %b and %B.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
diff --git a/src/interfaces/ecpg/pgtypeslib/timestamp.c b/src/interfaces/ecpg/pgtypeslib/timestamp.c
index 810dd06ee6..ddb82a1ad6 100644
--- a/src/interfaces/ecpg/pgtypeslib/timestamp.c
+++ b/src/interfaces/ecpg/pgtypeslib/timestamp.c
@@ -334,13 +334,13 @@ dttofmtasc_replace(timestamp * ts, date dDate, int dow, struct tm *tm,
/* XXX should be locale aware */
case 'b':
case 'h':
- replace_val.str_val = months[tm->tm_mon];
+ replace_val.str_val = months[tm->tm_mon - 1];
replace_type = PGTYPES_TYPE_STRING_CONSTANT;
break;
/* the full name of the month */
/* XXX should be locale aware */
case 'B':
- replace_val.str_val = pgtypes_date_months[tm->tm_mon];
+ replace_val.str_val = pgtypes_date_months[tm->tm_mon - 1];
replace_type = PGTYPES_TYPE_STRING_CONSTANT;
break;
Attachments:
[text/plain] ecpg-month-index-fix.patch (845B, ../../20191129203832.xybuswyoa455mwy2@development/2-ecpg-month-index-fix.patch)
download | inline diff:
diff --git a/src/interfaces/ecpg/pgtypeslib/timestamp.c b/src/interfaces/ecpg/pgtypeslib/timestamp.c
index 810dd06ee6..ddb82a1ad6 100644
--- a/src/interfaces/ecpg/pgtypeslib/timestamp.c
+++ b/src/interfaces/ecpg/pgtypeslib/timestamp.c
@@ -334,13 +334,13 @@ dttofmtasc_replace(timestamp * ts, date dDate, int dow, struct tm *tm,
/* XXX should be locale aware */
case 'b':
case 'h':
- replace_val.str_val = months[tm->tm_mon];
+ replace_val.str_val = months[tm->tm_mon - 1];
replace_type = PGTYPES_TYPE_STRING_CONSTANT;
break;
/* the full name of the month */
/* XXX should be locale aware */
case 'B':
- replace_val.str_val = pgtypes_date_months[tm->tm_mon];
+ replace_val.str_val = pgtypes_date_months[tm->tm_mon - 1];
replace_type = PGTYPES_TYPE_STRING_CONSTANT;
break;
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
@ 2019-11-29 20:42 Tom Lane <tgl@sss.pgh.pa.us>
parent: PG Bug reporting form <noreply@postgresql.org>
1 sibling, 0 replies; 6+ messages in thread
From: Tom Lane @ 2019-11-29 20:42 UTC (permalink / raw)
To: paul@intekon.com; +Cc: pgsql-bugs@lists.postgresql.org
PG Bug reporting form <noreply@postgresql.org> writes:
> PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format
> specifier %b is used.
Yeah, you're clearly right. Defining struct pg_tm's tm_mon differently
from the common understanding of struct tm's tm_mon may not have been
the greatest idea we ever had :-(. I dug through the uses of tm_mon in
the rest of ecpglib and didn't find any other similar mistakes, though.
Will fix, thanks for the report!
regards, tom lane
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
@ 2019-11-29 20:43 Tom Lane <tgl@sss.pgh.pa.us>
parent: Tomas Vondra <tomas.vondra@2ndquadrant.com>
0 siblings, 1 reply; 6+ messages in thread
From: Tom Lane @ 2019-11-29 20:43 UTC (permalink / raw)
To: Tomas Vondra <tomas.vondra@2ndquadrant.com>; +Cc: paul@intekon.com; pgsql-bugs@lists.postgresql.org
Tomas Vondra <tomas.vondra@2ndquadrant.com> writes:
> The attached patch should fix this, I believe - both for %b and %B.
Yeah, I'd just come to the same conclusion. You're welcome to
do the commits, of course.
regards, tom lane
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
@ 2019-11-29 21:36 Tomas Vondra <tomas.vondra@2ndquadrant.com>
parent: Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 6+ messages in thread
From: Tomas Vondra @ 2019-11-29 21:36 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: paul@intekon.com; pgsql-bugs@lists.postgresql.org
On Fri, Nov 29, 2019 at 03:43:44PM -0500, Tom Lane wrote:
>Tomas Vondra <tomas.vondra@2ndquadrant.com> writes:
>> The attached patch should fix this, I believe - both for %b and %B.
>
>Yeah, I'd just come to the same conclusion. You're welcome to
>do the commits, of course.
>
OK, I'll commit shortly.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used.
@ 2019-11-30 14:21 Tomas Vondra <tomas.vondra@2ndquadrant.com>
parent: Tomas Vondra <tomas.vondra@2ndquadrant.com>
0 siblings, 0 replies; 6+ messages in thread
From: Tomas Vondra @ 2019-11-30 14:21 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: paul@intekon.com; pgsql-bugs@lists.postgresql.org
On Fri, Nov 29, 2019 at 10:36:14PM +0100, Tomas Vondra wrote:
>On Fri, Nov 29, 2019 at 03:43:44PM -0500, Tom Lane wrote:
>>Tomas Vondra <tomas.vondra@2ndquadrant.com> writes:
>>>The attached patch should fix this, I believe - both for %b and %B.
>>
>>Yeah, I'd just come to the same conclusion. You're welcome to
>>do the commits, of course.
>>
>
>OK, I'll commit shortly.
>
OK, pushed with backpatch to all supported branches. Turns out this is a
pretty ancient bug - it's there since at least 2003, I stopped digging
after that.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 6+ messages in thread
end of thread, other threads:[~2019-11-30 14:21 UTC | newest]
Thread overview: 6+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2019-11-29 19:40 BUG #16143: PGTYPEStimestamp_fmt_asc() returns the incorrect month when the format specifier %b is used. PG Bug reporting form <noreply@postgresql.org>
2019-11-29 20:38 ` Tomas Vondra <tomas.vondra@2ndquadrant.com>
2019-11-29 20:43 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-11-29 21:36 ` Tomas Vondra <tomas.vondra@2ndquadrant.com>
2019-11-30 14:21 ` Tomas Vondra <tomas.vondra@2ndquadrant.com>
2019-11-29 20:42 ` Tom Lane <tgl@sss.pgh.pa.us>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox