Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(19)

Issue 325001: Fix some error-case leaks in Mac breakpad code

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 11 months ago by stuart.morgan
Modified:
12 years, 11 months ago
Reviewers:
Mark Mentovai
Base URL:
http://google-breakpad.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Found by Xcode 4.2's Analyze.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
src/client/mac/handler/minidump_generator.cc View 1 chunk +7 lines, -2 lines 0 comments Download
src/client/mac/sender/uploader.m View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3
stuart.morgan
12 years, 11 months ago #1
Mark Mentovai
LGTM
12 years, 11 months ago #2
stuart.morgan
12 years, 11 months ago #3
(Updated description since Ben already fixed the leak it referred to in another
CL)
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1004:630ec63f810e-tainted