You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2025/04/11 20:13:10 UTC
(commons-compress) 14/21: Reduce boilerplate and some possible NullPointerExceptions
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git
commit f082047bda17524929a39d0def03dd17e4b9ab42
Author: Gary D. Gregory <ga...@gmail.com>
AuthorDate: Fri Apr 11 15:43:46 2025 -0400
Reduce boilerplate and some possible NullPointerExceptions
---
.../org/apache/commons/compress/archivers/sevenz/DeltaDecoder.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/main/java/org/apache/commons/compress/archivers/sevenz/DeltaDecoder.java b/src/main/java/org/apache/commons/compress/archivers/sevenz/DeltaDecoder.java
index 0b12298de..c7fd8ba8b 100644
--- a/src/main/java/org/apache/commons/compress/archivers/sevenz/DeltaDecoder.java
+++ b/src/main/java/org/apache/commons/compress/archivers/sevenz/DeltaDecoder.java
@@ -22,6 +22,7 @@
import java.io.InputStream;
import java.io.OutputStream;
+import org.apache.commons.lang3.ArrayUtils;
import org.tukaani.xz.DeltaOptions;
import org.tukaani.xz.FinishableWrapperOutputStream;
import org.tukaani.xz.UnsupportedOptionsException;
@@ -54,7 +55,7 @@ byte[] getOptionsAsProperties(final Object options) {
}
private int getOptionsFromCoder(final Coder coder) {
- if (coder.properties == null || coder.properties.length == 0) {
+ if (coder == null || ArrayUtils.isEmpty(coder.properties)) {
return 1;
}
return (0xff & coder.properties[0]) + 1;