Fix #676

pdfcpu · Aug 21, 2023 · b42e6eb · b42e6eb
1 parent 91b6234
commit b42e6eb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/pkg/pdfcpu/model/nameTree.go b/pkg/pdfcpu/model/nameTree.go
@@ -224,12 +224,12 @@ func (n *Node) HandleLeaf(xRefTable *XRefTable, k string, v types.Object, m Name
 			if err == nil {
 				break
 			}
-			if len(m) == 0 {
-				return err
-			}
 			if err != errNameTreeDuplicateKey {
 				return err
 			}
+			if len(m) == 0 {
+				return nil
+			}
 			kNew := k + "\x01"
 			dd, ok := m[kOrig]
 			if !ok {

diff --git a/pkg/pdfcpu/model/nameTree_test.go b/pkg/pdfcpu/model/nameTree_test.go
@@ -170,8 +170,8 @@ func destroyNameTree(t *testing.T, r *Node) {
 		t.Fatal("should not find Value for x")
 	}
 
-	if err := r.Add(nil, "c", types.StringLiteral("cvv"), nil, nil); err == nil {
-		t.Fatalf("update c:should trigger DuplicateKeyException")
+	if err := r.Add(nil, "c", types.StringLiteral("cvv"), nil, nil); err != nil {
+		t.Fatalf("update c:should not trigger DuplicateKeyException")
 	}
 
 	empty, ok, _ = r.Remove(nil, "c")