Fix #1111

hhrutter · hhrutter · commit f11409b30b20 · 2025-04-21T23:13:06.000+02:00
diff --git a/pkg/pdfcpu/validate/nameTree.go b/pkg/pdfcpu/validate/nameTree.go
@@ -719,7 +719,7 @@ func validateNameTree(xRefTable *model.XRefTable, name string, d types.Dict, roo
 			return "", "", nil, err
 		}
 
-		if a == nil {
+		if len(a) == 0 {
 			return "", "", nil, errors.New("pdfcpu: validateNameTree: missing \"Kids\" array")
 		}
 
@@ -734,7 +734,11 @@ func validateNameTree(xRefTable *model.XRefTable, name string, d types.Dict, roo
 			var kidNode *model.Node
 			kminKid, kmax, kidNode, err = validateNameTree(xRefTable, name, d, false)
 			if err != nil {
-				return "", "", nil, err
+				if xRefTable.ValidationMode == model.ValidationStrict {
+					return "", "", nil, err
+				}
+				continue
+				//return kmin, kmax, nil, nil
 			}
 			if kmin == "" {
 				kmin = kminKid
diff --git a/pkg/pdfcpu/validate/xReftable.go b/pkg/pdfcpu/validate/xReftable.go
@@ -250,16 +250,15 @@ func validateNames(xRefTable *model.XRefTable, rootDict types.Dict, required boo
 			continue
 		}
 
-		d1.Insert(treeName, value)
-
 		_, _, tree, err := validateNameTree(xRefTable, treeName, d, true)
 		if err != nil {
 			return err
 		}
 
-		if tree != nil {
+		if tree != nil && tree.Kmin != "" && tree.Kmax != "" {
 			// Internalize.
 			xRefTable.Names[treeName] = tree
+			d1.Insert(treeName, value)
 		}
 
 	}

Original file line number	Diff line number	Diff line change
`@@ -719,7 +719,7 @@ func validateNameTree(xRefTable *model.XRefTable, name string, d types.Dict, roo`
`719`	`719`	`return "", "", nil, err`
`720`	`720`	`}`
`721`	`721`
`722`		`- if a == nil {`
	`722`	`+ if len(a) == 0 {`
`723`	`723`	`return "", "", nil, errors.New("pdfcpu: validateNameTree: missing \"Kids\" array")`
`724`	`724`	`}`
`725`	`725`
`@@ -734,7 +734,11 @@ func validateNameTree(xRefTable *model.XRefTable, name string, d types.Dict, roo`
`734`	`734`	`var kidNode *model.Node`
`735`	`735`	`kminKid, kmax, kidNode, err = validateNameTree(xRefTable, name, d, false)`
`736`	`736`	`if err != nil {`
`737`		`- return "", "", nil, err`
	`737`	`+ if xRefTable.ValidationMode == model.ValidationStrict {`
	`738`	`+ return "", "", nil, err`
	`739`	`+ }`
	`740`	`+ continue`
	`741`	`+ //return kmin, kmax, nil, nil`
`738`	`742`	`}`
`739`	`743`	`if kmin == "" {`
`740`	`744`	`kmin = kminKid`
Original file line number	Diff line number	Diff line change
`@@ -250,16 +250,15 @@ func validateNames(xRefTable *model.XRefTable, rootDict types.Dict, required boo`
`250`	`250`	`continue`
`251`	`251`	`}`
`252`	`252`
`253`		`- d1.Insert(treeName, value)`
`254`		`-`
`255`	`253`	`_, _, tree, err := validateNameTree(xRefTable, treeName, d, true)`
`256`	`254`	`if err != nil {`
`257`	`255`	`return err`
`258`	`256`	`}`
`259`	`257`
`260`		`- if tree != nil {`
	`258`	`+ if tree != nil && tree.Kmin != "" && tree.Kmax != "" {`
`261`	`259`	`// Internalize.`
`262`	`260`	`xRefTable.Names[treeName] = tree`
	`261`	`+ d1.Insert(treeName, value)`
`263`	`262`	`}`
`264`	`263`
`265`	`264`	`}`