]> git.mxchange.org Git - simgear.git/commit
Patch from Melchior Franz:
authordavid <david>
Mon, 25 Mar 2002 19:50:32 +0000 (19:50 +0000)
committerdavid <david>
Mon, 25 Mar 2002 19:50:32 +0000 (19:50 +0000)
commita8e0002a64b13788855c4545afaa061bdab463fc
treebc09b3167ffb189fbd3a5d861910e2ff7a6a44a7
parentbf75cf72253e28fec7e8ff7430f8fd51e4ff0b26
Patch from Melchior Franz:

at several places material was copied to "buffer" using strncpy
without adding a closing '\0'. This again lead to access to non
initialized memory and potentially (and actually at least in one
case) to feeding garbage to atof(). In case the following garbage
happened to start with digits, we would get funny time
values.  :-)
   I just added the obligatory "buffer[n] = 0", which doesn't
really look professional now. Maybe we should use the string
class or define a helper function that strncopies =and= adds
a trailing zero?
   The last hunk fixes another buglet, that wasn't dangerous
at all, but caused an error message. The loop that should cut
the string at hash marks ('#') did neither stop at such, nor at
string ends. It always scanned the whole 256 character long
buffer and accessed uninitialized memory. valgrind doesn't
like that. I dropped the 256 counter, because fgets =does=
add the closing zero. It is safe to scan until we either
get the zero or the hash mark.
simgear/timing/timezone.cxx