Author: delphij
Date: Fri Sep 25 01:17:24 2015
New Revision: 288205
URL: https://svnweb.freebsd.org/changeset/base/288205

Log:
  MFC r287650:
  
  Use strlcpy() in favor of strncpy() as it's defined to have a nul character
  at the end of string buffer, and the code context do expects this to behave
  correctly (e.g. strchr).

Modified:
  stable/10/usr.sbin/gstat/gstat.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/usr.sbin/gstat/gstat.c
==============================================================================
--- stable/10/usr.sbin/gstat/gstat.c    Fri Sep 25 01:05:44 2015        
(r288204)
+++ stable/10/usr.sbin/gstat/gstat.c    Fri Sep 25 01:17:24 2015        
(r288205)
@@ -124,7 +124,7 @@ main(int argc, char **argv)
                        if (regcomp(&f_re, optarg, REG_EXTENDED) != 0)
                                errx(EX_USAGE,
                                    "Invalid filter - see re_format(7)");
-                       strncpy(f_s, optarg, sizeof(f_s));
+                       strlcpy(f_s, optarg, sizeof(f_s));
                        break;
                case 'o':
                        flag_o = 1;
@@ -216,7 +216,7 @@ main(int argc, char **argv)
                                getyx(stdscr, cury, curx);
                                getmaxyx(stdscr, maxy, maxx);
                        }
-                       strncpy(pf_s, f_s, sizeof(pf_s));
+                       strlcpy(pf_s, f_s, sizeof(pf_s));
                        max_flen = maxx - curx - 1;
                        if ((int)strlen(f_s) > max_flen && max_flen >= 0) {
                                if (max_flen > 3)
@@ -406,7 +406,7 @@ main(int argc, char **argv)
                                        err(1, "el_gets");
                                if (line_len > 1)
                                        history(hist, &hist_ev, H_ENTER, line);
-                               strncpy(tmp_f_s, line, sizeof(f_s));
+                               strlcpy(tmp_f_s, line, sizeof(f_s));
                                if ((p = strchr(tmp_f_s, '\n')) != NULL)
                                        *p = '\0';
                                /*
@@ -423,7 +423,7 @@ main(int argc, char **argv)
                                        refresh();
                                        sleep(1);
                                } else {
-                                       strncpy(f_s, tmp_f_s, sizeof(f_s));
+                                       strlcpy(f_s, tmp_f_s, sizeof(f_s));
                                        f_re = tmp_f_re;
                                }
                                break;
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to