From 9a3a9e3010cd959b6bdad5e7987366c9afcc3d21 Mon Sep 17 00:00:00 2001
From: Jon Trulson <jon@radscan.com>
Date: Thu, 5 Apr 2018 15:44:37 -0600
Subject: [PATCH] Revert "dtfile: coverity CIDs 88363,88405,89140,89612;
 insecure readlink"

This reverts commit 7fa35c29c4559ce8b4ca4d4b5f0216a5305bb6ea.
---
 cde/programs/dtfile/Directory.c     | 4 ++--
 cde/programs/dtfile/SharedProcs.c   | 2 +-
 cde/programs/dtfile/dtcopy/fsrtns.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/cde/programs/dtfile/Directory.c b/cde/programs/dtfile/Directory.c
index f8ed2fb4..578e9266 100644
--- a/cde/programs/dtfile/Directory.c
+++ b/cde/programs/dtfile/Directory.c
@@ -825,7 +825,7 @@ ReadFileData(
    {
      while ((link_len = readlink(link_file_name, link_path, MAX_PATH)) > 0)
      {
-       link_path[link_len-1] = 0;
+       link_path[link_len] = '\0';
        link_list = (char **)XtRealloc((char *)link_list, sizeof(char *) *
                                       (link_count + 2));
 
@@ -1071,7 +1071,7 @@ ReadFileData2(
    {
      while ((link_len = readlink(link_file_name, link_path, MAX_PATH)) > 0)
      {
-       link_path[link_len - 1] = 0;
+       link_path[link_len] = NILL;
        link_list = (char **)XtRealloc((char *)link_list, sizeof(char *) *
                                       (link_count + 2));
 
diff --git a/cde/programs/dtfile/SharedProcs.c b/cde/programs/dtfile/SharedProcs.c
index eadcc2f9..48a21aff 100644
--- a/cde/programs/dtfile/SharedProcs.c
+++ b/cde/programs/dtfile/SharedProcs.c
@@ -236,7 +236,7 @@ _DtFollowLink (
 
    while ((link_len = readlink(file, link_path, MAXPATHLEN)) > 0)
    {
-      link_path[link_len-1] = 0;
+      link_path[link_len] = '\0';
 
       /* Force the link to be an absolute path, if necessary */
       if (link_path[0] != '/')
diff --git a/cde/programs/dtfile/dtcopy/fsrtns.c b/cde/programs/dtfile/dtcopy/fsrtns.c
index 4956d5ae..5532aca2 100644
--- a/cde/programs/dtfile/dtcopy/fsrtns.c
+++ b/cde/programs/dtfile/dtcopy/fsrtns.c
@@ -187,7 +187,7 @@ CopyLink(char *sourceP, char *targetP, int repl, struct stat *statP)
   } while (l < 0 && errno == EINTR);
   if (l < 0)
     return errno;
-  buf[l-1] = 0;
+  buf[l] = 0;
   if (symlink(buf, targetP) == 0)
     return 0;
   else if (errno != EEXIST || !repl)
-- 
2.25.1