Fix #772

hhrutter · hhrutter · commit 6ae90db69a46 · 2024-01-31T19:14:21.000+01:00
diff --git a/pkg/pdfcpu/validate/font.go b/pkg/pdfcpu/validate/font.go
@@ -135,11 +135,18 @@ func validateFontDescriptorPart1(xRefTable *model.XRefTable, d types.Dict, dictN
 		return err
 	}
 
-	_, err = validateNameEntry(xRefTable, d, dictName, "FontName", REQUIRED, model.V10, nil)
+	required := true
+	if xRefTable.ValidationMode == model.ValidationRelaxed {
+		required = false
+	}
+	_, err = validateNameEntry(xRefTable, d, dictName, "FontName", required, model.V10, nil)
 	if err != nil {
-		_, err = validateStringEntry(xRefTable, d, dictName, "FontName", REQUIRED, model.V10, nil)
+		_, err = validateStringEntry(xRefTable, d, dictName, "FontName", required, model.V10, nil)
 		if err != nil {
-			return err
+			if xRefTable.ValidationMode != model.ValidationRelaxed {
+				return err
+			}
+			reportSpecViolation(xRefTable, err)
 		}
 	}
 
diff --git a/pkg/pdfcpu/validate/page.go b/pkg/pdfcpu/validate/page.go
@@ -102,8 +102,12 @@ func validatePageContents(xRefTable *model.XRefTable, d types.Dict) (hasContents
 
 		}
 
-		if xRefTable.ValidationMode != model.ValidationRelaxed {
-			return false, errors.Errorf("validatePageContents: empty page content array detected")
+		if !hasContents {
+			err := errors.Errorf("validatePageContents: empty page content array detected")
+			if xRefTable.ValidationMode == model.ValidationStrict {
+				return false, err
+			}
+			reportSpecViolation(xRefTable, err)
 		}
 
 	default:
diff --git a/pkg/pdfcpu/validate/xReftable.go b/pkg/pdfcpu/validate/xReftable.go
@@ -31,6 +31,17 @@ import (
 	"github.com/pkg/errors"
 )
 
+func reportSpecViolation(xRefTable *model.XRefTable, err error) {
+	// TODO Apply across code base.
+	pre := fmt.Sprintf("digesting spec violation around obj#(%d)", xRefTable.CurObj)
+	if log.ValidateEnabled() {
+		log.CLI.Printf("%s: %v\n", pre, err)
+	}
+	if log.CLIEnabled() {
+		log.Validate.Printf("%s: %v\n", pre, err)
+	}
+}
+
 // XRefTable validates a PDF cross reference table obeying the validation mode.
 func XRefTable(xRefTable *model.XRefTable) error {
 	if log.InfoEnabled() {

Original file line number	Diff line number	Diff line change
`@@ -102,8 +102,12 @@ func validatePageContents(xRefTable *model.XRefTable, d types.Dict) (hasContents`
`102`	`102`
`103`	`103`	`}`
`104`	`104`
`105`		`- if xRefTable.ValidationMode != model.ValidationRelaxed {`
`106`		`- return false, errors.Errorf("validatePageContents: empty page content array detected")`
	`105`	`+ if !hasContents {`
	`106`	`+ err := errors.Errorf("validatePageContents: empty page content array detected")`
	`107`	`+ if xRefTable.ValidationMode == model.ValidationStrict {`
	`108`	`+ return false, err`
	`109`	`+ }`
	`110`	`+ reportSpecViolation(xRefTable, err)`
`107`	`111`	`}`
`108`	`112`
`109`	`113`	`default:`