Use strncpy instead of strcpy

Even though a user has to be stupid to trigger this, strncpy should be
used if not to much hassle :)

1 files changed, 3 insertions, 3 deletions

diff --git a/src/bin/e_utils.c b/src/bin/e_utils.c
index 97ad44a974..abbf726950 100644
--- a/src/bin/e_utils.c
+++ b/src/bin/e_utils.c
@@ -163,19 +163,19 @@ e_util_head_exec(int head, const char *cmd)
         /* yes it could overflow... but who will overflow DISPLAY eh? why? to
          * "exploit" your own applications running as you?
          */
-        strcpy(buf, penv_display);
+        strncpy(buf, penv_display, sizeof(buf));
         buf[p2 - penv_display + 1] = 0;
         snprintf(buf2, sizeof(buf2), "%i", head);
         strcat(buf, buf2);
      }
    else if (p1) /* "blah:x */
      {
-        strcpy(buf, penv_display);
+        strncpy(buf, penv_display, sizeof(buf));
         snprintf(buf2, sizeof(buf2), ".%i", head);
         strcat(buf, buf2);
      }
    else
-     strcpy(buf, penv_display);
+     strncpy(buf, penv_display, sizeof(buf));
 
    ok = 1;
    exe = ecore_exe_run(cmd, NULL);