Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dead initialization in parse_abbr function #87526

Closed
alexhenrie mannequin opened this issue Mar 2, 2021 · 1 comment
Closed

Dead initialization in parse_abbr function #87526

alexhenrie mannequin opened this issue Mar 2, 2021 · 1 comment
Labels
stdlib Python modules in the Lib dir type-feature A feature request or enhancement

Comments

@alexhenrie
Copy link
Mannequin

alexhenrie mannequin commented Mar 2, 2021

BPO 43360
Nosy @alexhenrie
PRs
  • gh-87526: Remove dead initialization from parse_abbr function #24700
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = None
    created_at = <Date 2021-03-02.02:25:12.944>
    labels = ['type-feature', 'library', '3.10']
    title = 'Dead initialization in parse_abbr function'
    updated_at = <Date 2021-03-02.02:26:22.031>
    user = 'https://github.com/alexhenrie'

    bugs.python.org fields:

    activity = <Date 2021-03-02.02:26:22.031>
    actor = 'alex.henrie'
    assignee = 'none'
    closed = False
    closed_date = None
    closer = None
    components = ['Library (Lib)']
    creation = <Date 2021-03-02.02:25:12.944>
    creator = 'alex.henrie'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 43360
    keywords = ['patch']
    message_count = 1.0
    messages = ['387895']
    nosy_count = 1.0
    nosy_names = ['alex.henrie']
    pr_nums = ['24700']
    priority = 'normal'
    resolution = None
    stage = 'patch review'
    status = 'open'
    superseder = None
    type = 'enhancement'
    url = 'https://bugs.python.org/issue43360'
    versions = ['Python 3.10']

    @alexhenrie
    Copy link
    Mannequin Author

    alexhenrie mannequin commented Mar 2, 2021

    The parse_abbr function in _zoneinfo.c currently initializes the variable buff regardless of whether it is going to be used, and then initializes it again in the one path that does use it. The first initialization could be removed to simplify the code and clarify its intent.

    Defect identified by scan-build <https://clang-analyzer.llvm.org/scan-build.html\>

    @alexhenrie alexhenrie mannequin added 3.10 only security fixes stdlib Python modules in the Lib dir type-feature A feature request or enhancement labels Mar 2, 2021
    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    @iritkatriel iritkatriel removed the 3.10 only security fixes label Apr 5, 2023
    carljm added a commit to carljm/cpython that referenced this issue May 12, 2023
    * main:
      pythongh-91896: Fixup some docs issues following ByteString deprecation (python#104422)
      pythonGH-104371: check return value of calling `mv.release` (python#104417)
      pythongh-104415: Fix refleak tests for `typing.ByteString` deprecation (python#104416)
      pythonGH-86275: Implementation of hypothesis stubs for property-based tests, with zoneinfo tests (python#22863)
      pythonGH-103082: Filter LINE events in VM, to simplify tool implementation. (pythonGH-104387)
      pythongh-93649: Split gc- and allocation tests from _testcapimodule.c (pythonGH-104403)
      pythongh-104389: Add 'unused' keyword to Argument Clinic C converters (python#104390)
      pythongh-101819: Prepare _io._IOBase for module state (python#104386)
      pythongh-104413: Fix refleak when super attribute throws AttributeError (python#104414)
      Fix refleak in `super_descr_get` (python#104408)
      pythongh-87526: Remove dead initialization from _zoneinfo parse_abbr() (python#24700)
      pythongh-91896: Improve visibility of `ByteString` deprecation warnings (python#104294)
      pythongh-104371: Fix calls to `__release_buffer__` while an exception is active (python#104378)
      pythongh-104377: fix cell in comprehension that is free in outer scope (python#104394)
      pythongh-104392: Remove _paramspec_tvars from typing (python#104393)
      pythongh-104396: uuid.py to skip platform check for emscripten and wasi (pythongh-104397)
      pythongh-99108: Refresh HACL* from upstream (python#104401)
      pythongh-104301: Allow leading whitespace in disambiguated pdb statements (python#104342)
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    stdlib Python modules in the Lib dir type-feature A feature request or enhancement
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants