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

Issue 6695025: CHROMIUM: qcusbnet: don't leak clients_lock (Closed)

Created:
9 years, 9 months ago by Elly Fong-Jones
Modified:
9 years, 4 months ago
Reviewers:
Jason Glasgow
CC:
chromium-os-reviews_chromium.org, vb+kernel_google.com, Olof Johansson, sleffler+cc_chromium.org, msb+croskernel_chromium.org
Visibility:
Public.

Description

CHROMIUM: qcusbnet: don't leak clients_lock Was present in the original qualcomm driver; we reported it to them and they fixed it, but we didn't pull the fix into the rewrite. Oops. BUG=None TEST=Adhoc 'gobi3k reset' with a gobi3k present triggers this most of the time. Change-Id: I02bd90fd17bec841a8d448507e2f8117f57e8328 Signed-off-by: Elly Jones <ellyjones@chromium.org>; Signed-off-by: Jason Glasgow <jglasgow@chromium.org>; Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=fada8ab

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M drivers/net/usb/gobi/qmidevice.c View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Elly Fong-Jones
9 years, 9 months ago (2011-03-15 17:35:00 UTC) #1
Jason Glasgow
9 years, 9 months ago (2011-03-15 17:43:16 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698