Use %w instead of %v in error wrapping to preserve underlying errors in WritePacket

siavashta · siavashta · commit 4148babd8bb3 · 2025-09-05T14:57:38.000-07:00
diff --git a/pcapgo/write.go b/pcapgo/write.go
@@ -122,7 +122,7 @@ func (w *Writer) WritePacket(ci gopacket.CaptureInfo, data []byte) error {
 		return fmt.Errorf("invalid capture info %+v:  capture length > length", ci)
 	}
 	if err := w.writePacketHeader(ci); err != nil {
-		return fmt.Errorf("error writing packet header: %v", err)
+		return fmt.Errorf("error writing packet header: %w", err)
 	}
 	_, err := w.w.Write(data)
 	return err

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ func (w *Writer) WritePacket(ci gopacket.CaptureInfo, data []byte) error {`
`122`	`122`	`return fmt.Errorf("invalid capture info %+v: capture length > length", ci)`
`123`	`123`	`}`
`124`	`124`	`if err := w.writePacketHeader(ci); err != nil {`
`125`		`- return fmt.Errorf("error writing packet header: %v", err)`
	`125`	`+ return fmt.Errorf("error writing packet header: %w", err)`
`126`	`126`	`}`
`127`	`127`	`_, err := w.w.Write(data)`
`128`	`128`	`return err`