| Summary: | NULL dereference on an error code path | ||
|---|---|---|---|
| Product: | dbus | Reporter: | Sviatoslav Chagaev <sviatoslav.chagaev> |
| Component: | core | Assignee: | Havoc Pennington <hp> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | major | ||
| Priority: | medium | Keywords: | patch |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: |
Patch for 1.6
Patch for 1.7 |
||
|
Description
Sviatoslav Chagaev
2013-09-13 17:01:29 UTC
Created attachment 85779 [details] [review] Patch for 1.6 Created attachment 85780 [details] [review] Patch for 1.7 Looks correct, although I hate this style of cleanup. I'll propose a patch to make the cleanup more obvious... (In reply to comment #3) > Looks correct, although I hate this style of cleanup. I'll propose a patch > to make the cleanup more obvious... ... actually, I won't, because the DBusString API is remarkably annoying. Never mind :-( Fixed in git for 1.6.16 and 1.7.6, thanks. (In reply to comment #4) > (In reply to comment #3) > > Looks correct, although I hate this style of cleanup. I'll propose a patch > > to make the cleanup more obvious... > > ... actually, I won't, because the DBusString API is remarkably annoying. > Never mind :-( > > Fixed in git for 1.6.16 and 1.7.6, thanks. Thanks! |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.