fix: check for nil pointers in PageLog comparison

robinovitch61 · robinovitch61 · commit 3144457776c1 · 2025-03-29T12:12:58.000-07:00
diff --git a/internal/model/page_log.go b/internal/model/page_log.go
@@ -74,6 +74,12 @@ func (l PageLog) Equals(other interface{}) bool {
 	if !ok {
 		return false
 	}
+	if l.Log == nil || otherLog.Log == nil {
+		return false
+	}
+	if l.Timestamps == nil || otherLog.Timestamps == nil {
+		return false
+	}
 	return l.Log.LineBuffer.Content() == otherLog.Log.LineBuffer.Content() && l.Timestamps.Full == otherLog.Timestamps.Full
 }
 

Original file line number	Diff line number	Diff line change
`@@ -74,6 +74,12 @@ func (l PageLog) Equals(other interface{}) bool {`
`74`	`74`	`if !ok {`
`75`	`75`	`return false`
`76`	`76`	`}`
	`77`	`+ if l.Log == nil \|\| otherLog.Log == nil {`
	`78`	`+ return false`
	`79`	`+ }`
	`80`	`+ if l.Timestamps == nil \|\| otherLog.Timestamps == nil {`
	`81`	`+ return false`
	`82`	`+ }`
`77`	`83`	`return l.Log.LineBuffer.Content() == otherLog.Log.LineBuffer.Content() && l.Timestamps.Full == otherLog.Timestamps.Full`
`78`	`84`	`}`
`79`	`85`