Fixed possible NULL pointer dereference.

Thanks to Jinsheng Ba <bajinsheng@u.nus.edu> for the report and patch.
DCMTK · Sep 15, 2021 · 5c14bf5 · 5c14bf5
1 parent 79ddbd2
commit 5c14bf5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/dcmnet/libsrc/diutil.cc b/dcmnet/libsrc/diutil.cc
@@ -171,7 +171,8 @@ DU_getStringDOElement(DcmItem *obj, DcmTagKey t, char *s, size_t bufsize)
             s[0] = '\0';
         } else {
             ec =  elem->getString(aString);
-            OFStandard::strlcpy(s, aString, bufsize);
+            if (ec == EC_Normal)
+                OFStandard::strlcpy(s, aString, bufsize);
         }
     }
     return (ec == EC_Normal);