From f02ecf0235cb17c90e9d1d8c155bd64e729fa46e Mon Sep 17 00:00:00 2001 From: Jakob Heher Date: Thu, 16 Apr 2026 16:14:46 +0200 Subject: fix some semgrep reported issues (#83) --- .../gv/egiz/pdfas/common/utils/TempFileHelper.java | 32 ++++++++-------------- 1 file changed, 12 insertions(+), 20 deletions(-) (limited to 'pdf-as-common/src') diff --git a/pdf-as-common/src/main/java/at/gv/egiz/pdfas/common/utils/TempFileHelper.java b/pdf-as-common/src/main/java/at/gv/egiz/pdfas/common/utils/TempFileHelper.java index 0a1c0c1a..1d2096b9 100644 --- a/pdf-as-common/src/main/java/at/gv/egiz/pdfas/common/utils/TempFileHelper.java +++ b/pdf-as-common/src/main/java/at/gv/egiz/pdfas/common/utils/TempFileHelper.java @@ -30,6 +30,8 @@ import java.util.ArrayList; import java.util.List; import java.util.UUID; +import lombok.Lombok; +import lombok.SneakyThrows; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -46,15 +48,21 @@ public class TempFileHelper implements IProfileConstants { private String tmpDir = "tmp"; - private MessageDigest messageDigest = null; + private static final MessageDigest messageDigest; - private List tmpFiles = new ArrayList(); + static { + try { + messageDigest = MessageDigest.getInstance("SHA-256"); + } catch (NoSuchAlgorithmException e) { + throw Lombok.sneakyThrow(e); + } + } + + private List tmpFiles = new ArrayList(); private boolean needsDeletion = false; public TempFileHelper(ISettings settings) { - initializeMD(); - String myTmpDir = settings.getValue(TMP_DIR); if(myTmpDir != null) { File myTmpDirFile = new File(myTmpDir); @@ -106,22 +114,6 @@ public class TempFileHelper implements IProfileConstants { } } - private void initializeMD() { - try { - messageDigest = MessageDigest.getInstance("SHA1"); - return; - } catch (NoSuchAlgorithmException e) { - logger.warn("SHA1 not available", e); - } - try { - messageDigest = MessageDigest.getInstance("MD5"); - return; - } catch (NoSuchAlgorithmException e) { - logger.warn("MD5 not available", e); - } - throw new RuntimeException("Need at least SHA1 or MD5 Message Digest, none available!"); - } - public void setTemporaryDirectory(String directory) { tmpDir = directory; -- cgit v1.2.3