Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(173)

Side by Side Diff: base/mac/scoped_mach_port.cc

Issue 11188003: Fix some potential Mach port leaks from mach_host_self using a new ScopedMachPort class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/mac/scoped_mach_port.h"
6
7 namespace base {
8 namespace mac {
9
10 ScopedMachPort::ScopedMachPort() : port_(MACH_PORT_NULL) {
11 }
12
13 ScopedMachPort::ScopedMachPort(mach_port_t port) : port_(port) {
14 }
15
16 ScopedMachPort::~ScopedMachPort() {
17 if (port_ != MACH_PORT_NULL) {
18 mach_port_deallocate(mach_task_self(), port_);
19 port_ = MACH_PORT_NULL;
Mark Mentovai 2012/10/16 16:34:28 Not strictly necessary for the destructor.
Robert Sesek 2012/10/16 16:40:23 Done.
20 }
21 }
22
23 ScopedMachPort::operator mach_port_t() const {
24 return port_;
25 }
26
27 mach_port_t ScopedMachPort::get() const {
28 return port_;
29 }
30
31 } // namespace mac
32 } // namespace base
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698