svn commit: r277934 - stable/9/usr.bin/sed

Pedro F. Giffuni pfg at FreeBSD.org
Fri Jan 30 14:38:00 UTC 2015


Author: pfg
Date: Fri Jan 30 14:37:59 2015
New Revision: 277934
URL: https://svnweb.freebsd.org/changeset/base/277934

Log:
  MFC	r277802, r277811:
  
  sed: Fix resource leak and dereference after NULL.
  
  process.c:
  Protect access against NULL.
  
  main.c:
  Prevent outfile overwrite resource leak.
  
  CID:	271181
  CID:	1006930
  
  Obtained from:	NetBSD

Modified:
  stable/9/usr.bin/sed/main.c
  stable/9/usr.bin/sed/process.c
Directory Properties:
  stable/9/usr.bin/sed/   (props changed)

Modified: stable/9/usr.bin/sed/main.c
==============================================================================
--- stable/9/usr.bin/sed/main.c	Fri Jan 30 14:36:02 2015	(r277933)
+++ stable/9/usr.bin/sed/main.c	Fri Jan 30 14:37:59 2015	(r277934)
@@ -404,6 +404,8 @@ mf_fgets(SPACE *sp, enum e_spflag spflag
 			if (len >= sizeof(tmpfname))
 				errx(1, "%s: name too long", fname);
 			unlink(tmpfname);
+			if (outfile != NULL && outfile != stdout)
+				fclose(outfile);
 			if ((outfile = fopen(tmpfname, "w")) == NULL)
 				err(1, "%s", fname);
 			fchown(fileno(outfile), sb.st_uid, sb.st_gid);

Modified: stable/9/usr.bin/sed/process.c
==============================================================================
--- stable/9/usr.bin/sed/process.c	Fri Jan 30 14:36:02 2015	(r277933)
+++ stable/9/usr.bin/sed/process.c	Fri Jan 30 14:37:59 2015	(r277934)
@@ -314,7 +314,7 @@ applies(struct s_command *cp)
                                 } else
                                         r = 1;
                         }
-		} else if (MATCH(cp->a1)) {
+		} else if (cp->a1 && MATCH(cp->a1)) {
 			/*
 			 * If the second address is a number less than or
 			 * equal to the line number first selected, only


More information about the svn-src-stable-9 mailing list