From 4f01d3d53b71774ae11b512f1f3ca8ec8dc28aed Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Wed, 23 Jan 2013 19:34:47 -0500 Subject: [PATCH] vbox: Address false positive for NULL dereference Resolve a false positive from 'vboxIIDFromUUID_v2_x()'. The code sets 'iid->value = &iid->backing' unconditionally prior to calling 'nsIDFromChar()'. The 'vboxIIDUnalloc_v2_x()' checks iid->value to not be &iid->backing. The iid->backing is a static buffer within the initialized structure. --- src/vbox/vbox_tmpl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/vbox/vbox_tmpl.c b/src/vbox/vbox_tmpl.c index 2b3fa255c4..d2cd0b8c69 100644 --- a/src/vbox/vbox_tmpl.c +++ b/src/vbox/vbox_tmpl.c @@ -444,6 +444,7 @@ vboxIIDFromUUID_v2_x(vboxGlobalData *data, vboxIID_v2_x *iid, iid->value = &iid->backing; + sa_assert(iid->value); nsIDFromChar(iid->value, uuid); } -- 2.47.2