GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — [3.6] bpo-29110: Fix file object leak in `aifc.open` by methane · Pull Request #310 · python/cpython

Conversation

@methane

@methane methane commented Feb 26, 2017

Copy link
Copy Markdown
Member

cherry pick of GH-162 and GH-293

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Uberi and others added 2 commits February 26, 2017 21:29
@Uberi @methane
bpo-29110: Fix file object leak in aifc.open when given invalid AIF…
0a8c34e 
…F file. (pythonGH-162)

(cherry picked from commit 03f68b6)
@methane
bpo-29110: add test for Aifc_write. (pythonGH-293)
09d29fc 
follow up of pythonGH-162
(cherry picked from commit 5dc33ee)
@methane methane added the cherry-pick for 3.6 label Feb 26, 2017
@the-knights-who-say-ni the-knights-who-say-ni added the CLA signed label Feb 26, 2017
serhiy-storchaka
serhiy-storchaka approved these changes Feb 26, 2017
View reviewed changes
duboviy
duboviy approved these changes Feb 26, 2017
View reviewed changes
@methane
methane merged commit c9131b6 into python:3.6 Feb 26, 2017
@methane
methane deleted the fix-aifc-test-36 branch February 26, 2017 19:09
jaraco pushed a commit that referenced this pull request Dec 2, 2022
@dependabot-preview
Bump pytest from 5.3.4 to 5.3.5 (#310)
eed5e33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes
+1 more reviewer
@duboviy duboviy duboviy approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@methane @serhiy-storchaka @duboviy @the-knights-who-say-ni @Uberi
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.