[AIX][BigArchive] Treat the archive is empty if the first child member offset is...
authorKai Luo <lkail@cn.ibm.com>
Mon, 19 Dec 2022 05:01:55 +0000 (13:01 +0800)
committerKai Luo <lkail@cn.ibm.com>
Mon, 19 Dec 2022 05:18:46 +0000 (13:18 +0800)
If the archive contains free list and contains no member file, the buffer length doesn't equal to length of the header.

Reviewed By: Esme, DiggerLin, #powerpc

Differential Revision: https://reviews.llvm.org/D138986

llvm/include/llvm/Object/Archive.h
llvm/test/Object/archive-big-malformed-first-member.test [new file with mode: 0644]
llvm/test/Object/archive-big-read-empty-with-freelist.test

index f5b9fca..56019ee 100644 (file)
@@ -413,9 +413,7 @@ public:
   BigArchive(MemoryBufferRef Source, Error &Err);
   uint64_t getFirstChildOffset() const override { return FirstChildOffset; }
   uint64_t getLastChildOffset() const { return LastChildOffset; }
-  bool isEmpty() const override {
-    return Data.getBufferSize() == sizeof(FixLenHdr);
-  };
+  bool isEmpty() const override { return getFirstChildOffset() == 0; }
 };
 
 } // end namespace object
diff --git a/llvm/test/Object/archive-big-malformed-first-member.test b/llvm/test/Object/archive-big-malformed-first-member.test
new file mode 100644 (file)
index 0000000..9107bdb
--- /dev/null
@@ -0,0 +1,4 @@
+# Test reading an empty archive with first member's offset is not zero.
+# RUN: echo    "<bigaf>" > %t.a
+# RUN: echo -n "0                   0                   0                   128                 0                   0                   " >> %t.a
+# RUN: not llvm-ar tv %t.a 2>&1 | grep 'truncated or malformed archive'
index 2ac3253..05e45ca 100644 (file)
@@ -1,3 +1,3 @@
 # Test reading an empty archive with free list in it.
-# RUN: not llvm-ar tv %p/Inputs/aix-empty-big-archive-with-freelist.a 2>&1 \
-# RUN:   | grep 'truncated or malformed archive'
+# RUN: llvm-ar tv %p/Inputs/aix-empty-big-archive-with-freelist.a 2>&1 \
+# RUN:   | not grep 'truncated or malformed archive'