1092875 - Use of uninitialized variables

Status: NEW

(Core :: DOM: Editor, defect, P5)

Description

[Mats Palmgren (inactive)]

Warning: -Wsometimes-uninitialized in editor/libeditor/nsHTMLEditRules.cpp: variable 'res' is used uninitialized whenever 'if' condition is true
editor/libeditor/nsHTMLEditRules.cpp:4952:7: warning: variable 'res' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
  if (!firstChild)
      ^~~~~~~~~~~
editor/libeditor/nsHTMLEditRules.cpp:4996:10: note: uninitialized use occurs here
  return res;
         ^~~
editor/libeditor/nsHTMLEditRules.cpp:4952:3: note: remove the 'if' if its condition is always false
  if (!firstChild)
  ^~~~~~~~~~~~~~~~
editor/libeditor/nsHTMLEditRules.cpp:4941:3: note: variable 'res' is declared here
  nsresult res;
  ^
In file included from editor/libeditor/Unified_cpp_editor_libeditor1.cpp:128:
Warning: -Wsometimes-uninitialized in editor/libeditor/nsHTMLEditorStyle.cpp: variable 'result' is used uninitialized whenever 'if' condition is false
editor/libeditor/nsHTMLEditorStyle.cpp:1120:7: warning: variable 'result' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
  if (range) {
      ^~~~~
editor/libeditor/nsHTMLEditorStyle.cpp:1275:10: note: uninitialized use occurs here
  return result;
         ^~~~~~
editor/libeditor/nsHTMLEditorStyle.cpp:1120:3: note: remove the 'if' if its condition is always true
  if (range) {
  ^~~~~~~~~~~
editor/libeditor/nsHTMLEditorStyle.cpp:1106:3: note: variable 'result' is declared here
  nsresult result;
  ^
2 warnings generated.

Comment 1

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority and severity.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.