ports/databases/db5/files/patch-lang_tcl_tcl__seq.c
Matthias Andree 7a68cdcef2 Avoid junk pointer when __db_vrfy_datapage() fails
Rather than second-guessing what the __os_free() might be doing and
avoiding it, initialize the pointer to NULL, which __os_free() will
skip.  This should be the safer approach if Oracle ever patches other
parts of db 5.3.

PR:		210829
Submitted by:	Mark Millard
2016-11-09 22:31:48 +00:00

16 lines
564 B
C

--- lang/tcl/tcl_seq.c.orig 2013-09-09 15:35:07 UTC
+++ lang/tcl/tcl_seq.c
@@ -555,9 +555,10 @@ tcl_SeqGetFlags(interp, objc, objv, seq)
for (i = 0; seq_flags[i].flag != 0; i++)
if (LF_ISSET(seq_flags[i].flag)) {
if (strlen(buf) > 0)
- (void)strncat(buf, " ", sizeof(buf));
- (void)strncat(
- buf, seq_flags[i].arg, sizeof(buf));
+ (void)strncat(buf, " ",
+ sizeof(buf) - (strlen(buf) + 1));
+ (void)strncat(buf, seq_flags[i].arg,
+ sizeof(buf) - (strlen(buf) + 1));
}
res = NewStringObj(buf, strlen(buf));