You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2026/10/26 20:09:30 UTC

(poi) 07/07: Fuzzing: Remove double testing of write()

This is an automated email from the ASF dual-hosted git repository.

centic9 pushed a commit to branch npes_fuzzing_and_test_improvements
in repository https://gitbox.apache.org/repos/asf/poi.git

commit d5104f33d2a72bc66d51ad4c68ec841fee9393af
Author: Dominik Stadler <ce...@apache.org>
AuthorDate: Mon Oct 26 10:16:42 2026 +0100

    Fuzzing: Remove double testing of write()
    
    Should be fully covered by the separate fuzzers already
---
 poi-fuzz/src/main/java/org/apache/poi/fuzz/POIFuzzer.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/poi-fuzz/src/main/java/org/apache/poi/fuzz/POIFuzzer.java b/poi-fuzz/src/main/java/org/apache/poi/fuzz/POIFuzzer.java
index 3318458eff..52fa443106 100644
--- a/poi-fuzz/src/main/java/org/apache/poi/fuzz/POIFuzzer.java
+++ b/poi-fuzz/src/main/java/org/apache/poi/fuzz/POIFuzzer.java
@@ -107,7 +107,7 @@ public class POIFuzzer {
 				}
 			}
 
-			wb.write(NullOutputStream.INSTANCE);
+			// writing is tested in the separate fuzzers anyway: wb.write(NullOutputStream.INSTANCE);
 		} catch (IOException | POIXMLException | IllegalArgumentException | RecordFormatException |
 				 IndexOutOfBoundsException | HSLFException | RecordInputStream.LeftoverDataException |
 				 IllegalStateException | BufferUnderflowException | OpenXML4JRuntimeException |


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org