sv_gets() did not NUL-terminate SV when reading records
authorGisle Aas <gisle@aas.no>
Fri, 17 Jul 1998 22:49:32 +0000 (00:49 +0200)
committerGurusamy Sarathy <gsar@cpan.org>
Sun, 19 Jul 1998 07:04:45 +0000 (07:04 +0000)
Message-ID: <m390lsb3tv.fsf@furu.g.aas.no>

p4raw-id: //depot/perl@1564

sv.c

diff --git a/sv.c b/sv.c
index 29c45fd..a54c6ba 100644 (file)
--- a/sv.c
+++ b/sv.c
@@ -3239,12 +3239,7 @@ sv_gets(register SV *sv, register PerlIO *fp, I32 append)
       /* Grab the size of the record we're getting */
       recsize = SvIV(SvRV(rs));
       (void)SvPOK_only(sv);    /* Validate pointer */
-      /* Make sure we've got the room to yank in the whole thing */
-      if (SvLEN(sv) <= recsize + 3) {
-        /* No, so make it bigger */
-        SvGROW(sv, recsize + 3);
-      }
-      buffer = SvPVX(sv); /* Get the location of the final buffer */
+      buffer = SvGROW(sv, recsize + 1);
       /* Go yank in */
 #ifdef VMS
       /* VMS wants read instead of fread, because fread doesn't respect */
@@ -3255,6 +3250,7 @@ sv_gets(register SV *sv, register PerlIO *fp, I32 append)
       bytesread = PerlIO_read(fp, buffer, recsize);
 #endif
       SvCUR_set(sv, bytesread);
+      buffer[bytesread] = '\0';
       return(SvCUR(sv) ? SvPVX(sv) : Nullch);
     }
     else if (RsPARA(rs)) {