bugfix(dfget) fix panic for slice bounds out of range when sendSuccessPiece

yourmoonlight · dave.geng · commit 06d5a14c6e96 · 2020-09-28T10:53:10.000+08:00
Signed-off-by: yourmoonlight &lt;foxkingmoonlight@gmail.com&gt;
diff --git a/dfget/core/downloader/p2p_downloader/client_writer.go b/dfget/core/downloader/p2p_downloader/client_writer.go
@@ -291,7 +291,7 @@ func sendSuccessPiece(api api.SupernodeAPI, cid string, piece *Piece, cost time.
 
 	if cost.Seconds() > 2.0 {
 		logrus.Infof(
-			"async writer and report suc from dst:%s... cost:%.3f for range:%s",
-			piece.DstCid[:25], cost.Seconds(), piece.Range)
+			"async writer and report suc from dst:%s cost:%.3f for range:%s",
+			piece.DstCid, cost.Seconds(), piece.Range)
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -291,7 +291,7 @@ func sendSuccessPiece(api api.SupernodeAPI, cid string, piece *Piece, cost time.`
`291`	`291`
`292`	`292`	`if cost.Seconds() > 2.0 {`
`293`	`293`	`logrus.Infof(`
`294`		`- "async writer and report suc from dst:%s... cost:%.3f for range:%s",`
`295`		`- piece.DstCid[:25], cost.Seconds(), piece.Range)`
	`294`	`+ "async writer and report suc from dst:%s cost:%.3f for range:%s",`
	`295`	`+ piece.DstCid, cost.Seconds(), piece.Range)`
`296`	`296`	`}`
`297`	`297`	`}`