Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 03 Jul 2016 21:05:09 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 210798] devel/gdb: compile-loc2c.c's do_compile_dwarf_expr_to_c( . . . ) uses uoffset += . . . with uoffset uninitialized as of -r417989; at least one other uninitialized example
Message-ID:  <bug-210798-13-1Sjv61DSMI@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-210798-13@https.bugs.freebsd.org/bugzilla/>
References:  <bug-210798-13@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D210798

--- Comment #3 from Mark Millard <markmi@dsl-only.net> ---
(In reply to Mark Millard from comment #0)

I probably should have been explicit that for the do_compile_dwarf_expr_to_c
code as it is:

push (indent, stream, uoffset);

always involves uoffset's uninitialized value.

If the "+=3D" were just "=3D" then the lack of a else assigning a value to =
uoffset
would still have a path in which uoffset involved an uninitialized value in=
 the
push call.

But it may be that an initialization of uoffset to something like 0 may take
care of things without an explicit else: the above wording is not trying to
point out the specific fix to make.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-210798-13-1Sjv61DSMI>