[issue18343] faulthandler_register_py() may use _Py_sighandler_t previous with initialization

[Christian Heimes]

New submission from Christian Heimes:

faulthandler_register_py() may use the variable previous uninitialized if user->enabled is set.

Excerpt from Coverity log:

var_decl: Declaring variable "previous" without initializer.
711    _Py_sighandler_t previous;

8. Condition "!user->enabled", taking false branch
738    if (!user->enabled) {

CID 984060 (#1 of 1): Uninitialized scalar variable (UNINIT)
12. uninit_use: Using uninitialized value "previous": field "previous"."sa_restorer" is uninitialized.
752    user->previous = previous;

----------
assignee: haypo
messages: 192154
nosy: christian.heimes, haypo
priority: normal
severity: normal
stage: needs patch
status: open
title: faulthandler_register_py() may use _Py_sighandler_t previous with initialization
type: behavior
versions: Python 3.3, Python 3.4

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue18343>
_______________________________________