Issue found by Svace static analyzer

[vvdvortsova]

Hello
This warning found with the Svace.
There is mb can be nil. It can lead to panic.

pdfcpu/pkg/pdfcpu/info.go

Line 254 in 281745f

mb := pb.MediaBox()

due to

pdfcpu/pkg/pdfcpu/boxes.go

Line 86 in 281745f

return nil

mb is dereferenced here

pdfcpu/pkg/pdfcpu/info.go

Line 256 in 281745f

if cb == nil || mb.equals(*cb) {

[hhrutter]

Hello!
Could you post the whole report, so things stay in context for a better solution.
Thank you for reporting this!

[vvdvortsova]

Yes, sure. The whole report looks like this

func appendPageBoxesInfo(...) {
...
mb := pb.MediaBox() // here mb may be nil
cb := pb.CropBox()
if cb == nil || mb.equals(*cb) {// here mb is dereferenced and it may lead to panic
      appendEqualMediaAndCropBoxInfo(ss, pb, unit, currUnit)
      return
}
appendNotEqualMediaAndCropBoxInfo(ss, pb, unit, currUnit)

func (pb PageBoundaries) MediaBox() *Rectangle { // may return nil value 
	if pb.Media == nil {
		return nil
	}
	return pb.Media.Rect
}