712508 - Clang Static Analysis: Garbage return value in security/manager/ssl/src/nsCertTree.cpp

Status: RESOLVED DUPLICATE of bug 550185

(Core :: Security, defect)

Description

[Christian Holler (:decoder)]

The following report (in the URL field) has been generated by static analysis using Clang.

It would be good if someone familiar with the particular code could check if

- this is really a bug or a false positive
- and/or if it makes sense to adjust the code (even if there is not a real bug present, e.g. by adding a missing initialization).


In this particular report, the problem seems to be that the | nsresult rv; | variable declared in line 1130 could be returned uninitialized (there exists at least one path through the code). It would probably be good to initialize this variable to a sane default value to be sure.

Comment 1

[:Cykesiopka]

Duplicate of Bug 550185 maybe?

Comment 2

[Dana Keeler (she/her) (use needinfo) [:keeler] (on leave)]

(In reply to Cykesiopka from comment #1)
> Duplicate of Bug 550185 maybe?

Looks like it.