Bug 99712 - test: Add some OOM paths to the dbus-daemon-eavesdrop path
Summary: test: Add some OOM paths to the dbus-daemon-eavesdrop path
Status: RESOLVED FIXED
Alias: None
Product: dbus
Classification: Unclassified
Component: core (show other bugs)
Version: unspecified
Hardware: All All
: medium normal
Assignee: D-Bus Maintainers
QA Contact: D-Bus Maintainers
URL:
Whiteboard: review+
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-02-08 10:12 UTC by Philip Withnall
Modified: 2017-02-13 15:58 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
test: Add some OOM paths to the dbus-daemon-eavesdrop path (2.54 KB, patch)
2017-02-08 10:12 UTC, Philip Withnall
Details | Splinter Review

Description Philip Withnall 2017-02-08 10:12:02 UTC
Another minor Coverity fix for test cases which would never fail in this way anyway.
Comment 1 Philip Withnall 2017-02-08 10:12:05 UTC
Created attachment 129411 [details] [review]
test: Add some OOM paths to the dbus-daemon-eavesdrop path

This will shut Coverity up. All the paths are checks on the return value
from dbus_message_set_destination().

Coverity IDs: 54822, 54823, 54824, 54825

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Comment 2 Simon McVittie 2017-02-13 15:58:30 UTC
Fixed in git for 1.11.10, 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.