diff options
| author | Christoph M. Becker <cmb@php.net> | 2016-07-29 01:27:14 +0200 |
|---|---|---|
| committer | Christoph M. Becker <cmb@php.net> | 2016-07-29 01:27:51 +0200 |
| commit | 09b5a453c1915b84b46e1c6e5687a2e1bc5bad22 (patch) | |
| tree | 38edcb53632a7f9116e4e1acfbd19e226d739236 /ext/calendar | |
| parent | 834919f0a56f3dc5d81be3ba228e046c026562c8 (diff) | |
| parent | 293dd3d7017e3df8f06898d3a7d1c00eb935ee0c (diff) | |
| download | php-git-09b5a453c1915b84b46e1c6e5687a2e1bc5bad22.tar.gz | |
Merge branch 'PHP-7.0' into PHP-7.1
Diffstat (limited to 'ext/calendar')
| -rw-r--r-- | ext/calendar/calendar.c | 20 | ||||
| -rw-r--r-- | ext/calendar/tests/bug71894.phpt | 32 |
2 files changed, 45 insertions, 7 deletions
diff --git a/ext/calendar/calendar.c b/ext/calendar/calendar.c index 1e4ad3dfde..76248a6554 100644 --- a/ext/calendar/calendar.c +++ b/ext/calendar/calendar.c @@ -420,15 +420,21 @@ PHP_FUNCTION(cal_from_jd) add_assoc_long(return_value, "year", year); /* day of week */ - dow = DayOfWeek(jd); - add_assoc_long(return_value, "dow", dow); - add_assoc_string(return_value, "abbrevdayname", DayNameShort[dow]); - add_assoc_string(return_value, "dayname", DayNameLong[dow]); + if (cal != CAL_JEWISH || year > 0) { + dow = DayOfWeek(jd); + add_assoc_long(return_value, "dow", dow); + add_assoc_string(return_value, "abbrevdayname", DayNameShort[dow]); + add_assoc_string(return_value, "dayname", DayNameLong[dow]); + } else { + add_assoc_null(return_value, "dow"); + add_assoc_string(return_value, "abbrevdayname", ""); + add_assoc_string(return_value, "dayname", ""); + } /* month name */ if(cal == CAL_JEWISH) { /* special case for Jewish calendar */ - add_assoc_string(return_value, "abbrevmonth", JEWISH_MONTH_NAME(year)[month]); - add_assoc_string(return_value, "monthname", JEWISH_MONTH_NAME(year)[month]); + add_assoc_string(return_value, "abbrevmonth", (year > 0 ? JEWISH_MONTH_NAME(year)[month] : "")); + add_assoc_string(return_value, "monthname", (year > 0 ? JEWISH_MONTH_NAME(year)[month] : "")); } else { add_assoc_string(return_value, "abbrevmonth", calendar->month_name_short[month]); add_assoc_string(return_value, "monthname", calendar->month_name_long[month]); @@ -741,7 +747,7 @@ PHP_FUNCTION(jdmonthname) break; case CAL_MONTH_JEWISH: /* jewish month */ SdnToJewish(julday, &year, &month, &day); - monthname = JEWISH_MONTH_NAME(year)[month]; + monthname = (year > 0 ? JEWISH_MONTH_NAME(year)[month] : ""); break; case CAL_MONTH_FRENCH: /* french month */ SdnToFrench(julday, &year, &month, &day); diff --git a/ext/calendar/tests/bug71894.phpt b/ext/calendar/tests/bug71894.phpt new file mode 100644 index 0000000000..ea2d6004e7 --- /dev/null +++ b/ext/calendar/tests/bug71894.phpt @@ -0,0 +1,32 @@ +--TEST-- +Bug #71894 (AddressSanitizer: global-buffer-overflow in zif_cal_from_jd) +--SKIPIF-- +<?php +if (!extension_loaded('calendar')) die('skip ext/calendar required'); +?> +--FILE-- +<?php +var_dump(cal_from_jd(347997, CAL_JEWISH)); +var_dump(jdmonthname(347997,CAL_MONTH_JEWISH));?> +--EXPECT-- +array(9) { + ["date"]=> + string(5) "0/0/0" + ["month"]=> + int(0) + ["day"]=> + int(0) + ["year"]=> + int(0) + ["dow"]=> + NULL + ["abbrevdayname"]=> + string(0) "" + ["dayname"]=> + string(0) "" + ["abbrevmonth"]=> + string(0) "" + ["monthname"]=> + string(0) "" +} +string(0) "" |