summaryrefslogtreecommitdiffstats
path: root/Modules/_zoneinfo.c
Commit message (Collapse)AuthorAgeFilesLines
* bpo-43132: Fix incorrect handling of PyObject_RichCompareBool() in _zoneinfo ↵Miss Islington (bot)2021-02-051-6/+17
| | | | | | | | | | | (GH-24450) (GH-24457) PyObject_RichCompareBool() returns -1 on error, but this case is not handled by the find_in_strong_cache() function. Any exception raised by PyObject_RichCompareBool() should be propagated. (cherry picked from commit effaec0bb54f381db8ccfa62514bc26b00946b40) Co-authored-by: Zackery Spytz <zspytz@gmail.com>
* bpo-42979: _zoneinfo exec function checks for PyDateTime_IMPORT failure ↵Miss Islington (bot)2021-01-271-0/+3
| | | | | | | | | | (GH-24333) (GH-24351) Importing datetime can fail. (cherry picked from commit eeb701adc0fc29ba803fddf133d917ff45639a00) Co-authored-by: Hai Shi <shihai1992@gmail.com> Co-authored-by: Hai Shi <shihai1992@gmail.com>
* bpo-43033: Fix the handling of PyObject_SetAttrString() in _zoneinfo.c ↵Miss Islington (bot)2021-01-271-1/+5
| | | | | | | | (GH-24345) (GH-24349) (cherry picked from commit 5327f370344a627f1578d8183d197feb286371c6) Co-authored-by: Zackery Spytz <zspytz@gmail.com>
* bpo-41995: Handle allocation failure in _tracemalloc and _zoneinfo (GH-22635)Miss Islington (bot)2021-01-201-0/+12
| | | | | (cherry picked from commit f1ff800db1f9fa5ff8f2fa2863796a46bfa9ee46) Co-authored-by: Yunlongs <lylgood@foxmail.com>
* bpo-40686: Fix compiler warnings on _zoneinfo.c (GH-23614) (GH-23804)Miss Islington (bot)2020-12-161-10/+4
| | | | | | | | | | | | | "uint8_t day" is unsigned and so "day < 0" test is always true. Remove the test to fix the following warnings on Windows: modules\_zoneinfo.c(1224): warning C4068: unknown pragma modules\_zoneinfo.c(1225): warning C4068: unknown pragma modules\_zoneinfo.c(1227): warning C4068: unknown pragma (cherry picked from commit aefb69b23f056c61e82ad228d950f348de090c70) Co-authored-by: Victor Stinner <vstinner@python.org> Co-authored-by: Victor Stinner <vstinner@python.org>
* bpo-41568: Fix refleaks in zoneinfo subclasses (GH-21907)Miss Islington (bot)2020-08-171-3/+3
| | | | | | | | | | | | | | | | | * Fix refleak in C module __init_subclass__ This was leaking a reference to the weak cache dictionary for every ZoneInfo subclass created. * Fix refleak in ZoneInfo subclass's clear_cache The previous version of the code accidentally cleared the global ZONEINFO_STRONG_CACHE variable (and inducing `ZoneInfo` to create a new strong cache) on calls to a subclass's `clear_cache()`. This would not affect guaranteed behavior, but it's still not the right thing to do (and it caused reference leaks). (cherry picked from commit c3dd7e45cc5d36bbe2295c2840faabb5c75d83e4) Co-authored-by: Paul Ganssle <paul@ganssle.io>
* bpo-41025: Fix subclassing for zoneinfo.ZoneInfo (GH-20965) (GH-21876)Miss Islington (bot)2020-08-141-1/+1
| | | | | | | | | | | Prior to this change, attempting to subclass the C implementation of zoneinfo.ZoneInfo gave the following error: TypeError: unbound method ZoneInfo.__init_subclass__() needs an argument https://bugs.python.org/issue41025 (cherry picked from commit 87d8287865e5c9f137f6b5cf8c34c2c509eb5e9d) Co-authored-by: Paul Ganssle <paul@ganssle.io>
* bpo-41336: Fix the error handling in zoneinfo_new_instance() (GH-21546)Miss Islington (bot)2020-07-201-1/+7
| | | | | | | Do not call PyObject_CallMethod() with a live exception (like KeyboardInterrupt). (cherry picked from commit eca2549f5a5048b44ca88b9555f1c62f094e3c12) Co-authored-by: Zackery Spytz <zspytz@gmail.com>
* Fix -Wstring-prototypes warnings in _zoneinfo.c. (GH-21478)Miss Islington (bot)2020-07-151-2/+2
| | | | | (cherry picked from commit 0108b2a2401d0ffffe7c07e5ab69a3b0c7593070) Co-authored-by: Benjamin Peterson <benjamin@python.org>
* [3.9] bpo-41056: Fix a possible MemoryError leak within zoneinfo. (GH-21007)Paul Ganssle2020-06-241-4/+2
| | | | | | | | This was detected by our Coverity scan as a REVERSE_INULL issue. Automerge-Triggered-By: @gpshead (cherry picked from commit d780fa7) Co-authored-by: Gregory P. Smith <greg@krypto.org>
* Fix compiler warnings in _zoneinfo.c (GH-20342)Miss Islington (bot)2020-05-271-30/+26
| | | | | | | | | | ``` D:\a\cpython\cpython\Modules\_zoneinfo.c(903,52): warning C4267: '=': conversion from 'size_t' to 'unsigned int', possible loss of data [D:\a\cpython\cpython\PCbuild\_zoneinfo.vcxproj] D:\a\cpython\cpython\Modules\_zoneinfo.c(904,44): warning C4267: '=': conversion from 'size_t' to 'unsigned int', possible loss of data [D:\a\cpython\cpython\PCbuild\_zoneinfo.vcxproj] D:\a\cpython\cpython\Modules\_zoneinfo.c(1772,31): warning C4244: '=': conversion from 'ssize_t' to 'uint8_t', possible loss of data [D:\a\cpython\cpython\PCbuild\_zoneinfo.vcxproj] ``` (cherry picked from commit e4799b95945b44eb0e2eea26473db8e0a49ed0ee) Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
* bpo-40705: Fix use-after-free in _zoneinfo's module_free (GH-20280)Miss Islington (bot)2020-05-241-6/+9
| | | | | (cherry picked from commit 06a1b8915d6674e40f0dccc422ca2c06212392d8) Co-authored-by: Ammar Askar <ammar@ammaraskar.com>
* [3.9] bpo-40714: Remove compile warning from _zoneinfo.c (GH-20291) (GH-20293)Miss Islington (bot)2020-05-211-2/+4
| | | | | | | | (cherry picked from commit a487a39dca4c41305928c7dfdbcb0b3aa344683b) Co-authored-by: Dong-hee Na <donghee.na92@gmail.com> Automerge-Triggered-By: @corona10
* bpo-40503: PEP 615: Tests and implementation for zoneinfo (GH-19909)Paul Ganssle2020-05-161-0/+2695
This is the initial implementation of PEP 615, the zoneinfo module, ported from the standalone reference implementation (see https://www.python.org/dev/peps/pep-0615/#reference-implementation for a link, which has a more detailed commit history). This includes (hopefully) all functional elements described in the PEP, but documentation is found in a separate PR. This includes: 1. A pure python implementation of the ZoneInfo class 2. A C accelerated implementation of the ZoneInfo class 3. Tests with 100% branch coverage for the Python code (though C code coverage is less than 100%). 4. A compile-time configuration option on Linux (though not on Windows) Differences from the reference implementation: - The module is arranged slightly differently: the accelerated module is `_zoneinfo` rather than `zoneinfo._czoneinfo`, which also necessitates some changes in the test support function. (Suggested by Victor Stinner and Steve Dower.) - The tests are arranged slightly differently and do not include the property tests. The tests live at test/test_zoneinfo/test_zoneinfo.py rather than test/test_zoneinfo.py or test/test_zoneinfo/__init__.py because we may do some refactoring in the future that would likely require this separation anyway; we may: - include the property tests - automatically run all the tests against both pure Python and C, rather than manually constructing C and Python test classes (similar to the way this works with test_datetime.py, which generates C and Python test cases from datetimetester.py). - This includes a compile-time configuration option on Linux (though not on Windows); added with much help from Thomas Wouters. - Integration into the CPython build system is obviously different from building a standalone zoneinfo module wheel. - This includes configuration to install the tzdata package as part of CI, though only on the coverage jobs. Introducing a PyPI dependency as part of the CI build was controversial, and this is seen as less of a major change, since the coverage jobs already depend on pip and PyPI. Additional changes that were introduced as part of this PR, most / all of which were backported to the reference implementation: - Fixed reference and memory leaks With much debugging help from Pablo Galindo - Added smoke tests ensuring that the C and Python modules are built The import machinery can be somewhat fragile, and the "seamlessly falls back to pure Python" nature of this module makes it so that a problem building the C extension or a failure to import the pure Python version might easily go unnoticed. - Adjustments to zoneinfo.__dir__ Suggested by Petr Viktorin. - Slight refactorings as suggested by Steve Dower. - Removed unnecessary if check on std_abbr Discovered this because of a missing line in branch coverage.