Fix #319

Author: hhrutter

pkg/pdfcpu/validate/action.go

@@ -26,9 +26,13 @@ import (
 func validateGoToActionDict(xRefTable *pdf.XRefTable, d pdf.Dict, dictName string) error {
 
 	// see 12.6.4.2 Go-To Actions
+	required := REQUIRED
+	if xRefTable.ValidationMode == pdf.ValidationRelaxed {
+		required = OPTIONAL
+	}
 
 	// D, required, name, byte string or array
-	return validateDestinationEntry(xRefTable, d, dictName, "D", REQUIRED, pdf.V10)
+	return validateDestinationEntry(xRefTable, d, dictName, "D", required, pdf.V10)
 }
 
 func validateGoToRActionDict(xRefTable *pdf.XRefTable, d pdf.Dict, dictName string) error {

pkg/pdfcpu/validate/objects.go

@@ -39,7 +39,7 @@ func validateEntry(xRefTable *pdf.XRefTable, d pdf.Dict, dictName, entryName str
 	o, found := d.Find(entryName)
 	if !found || o == nil {
 		if required {
-			return nil, errors.Errorf("dict=%s required entry=%s missing.", dictName, entryName)
+			return nil, errors.Errorf("dict=%s required entry=%s missing (obj#%d).", dictName, entryName, xRefTable.CurObj)
 		}
 		return nil, nil
 	}
@@ -51,13 +51,13 @@ func validateEntry(xRefTable *pdf.XRefTable, d pdf.Dict, dictName, entryName str
 
 	if o == nil {
 		if required {
-			return nil, errors.Errorf("dict=%s required entry=%s missing.", dictName, entryName)
+			return nil, errors.Errorf("dict=%s required entry=%s missing (obj#%d).", dictName, entryName, xRefTable.CurObj)
 		}
 		return nil, nil
 	}
 
 	// Version check
-	err = xRefTable.ValidateVersion(fmt.Sprintf("dict=%s entry=%s", dictName, entryName), sinceVersion)
+	err = xRefTable.ValidateVersion(fmt.Sprintf("dict=%s entry=%s (obj#%d)", dictName, entryName, xRefTable.CurObj), sinceVersion)
 	if err != nil {
 		return nil, err
 	}

pkg/pdfcpu/xreftable.go

@@ -127,6 +127,7 @@ type XRefTable struct {
 
 	// Validation
 	CurPage        int                       // current page during validation
+	CurObj         int                       // current object during validation, the last dereferenced object
 	ValidationMode int                       // see Configuration
 	ValidateLinks  bool                      // check for broken links in LinkAnnotations/URIDicts.
 	Valid          bool                      // true means successful validated against ISO 32000.
@@ -811,6 +812,8 @@ func (xRefTable *XRefTable) indRefToObject(ir *IndirectRef) (Object, error) {
 		return nil, nil
 	}
 
+	xRefTable.CurObj = int(ir.ObjectNumber)
+
 	// return dereferenced object
 	return entry.Object, nil
 }