src: remove unnecessary cast in crypto_sig.cc

tniessen · danielleadams · commit 2c44730d9725 · 2022-08-10T21:08:30.000-04:00
ByteSource::Allocated accepts a void pointer now, so we do not need to cast the argument to a char pointer. Refs: #43202 PR-URL: #44001 Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Feng Yu <F3n67u@outlook.com>
diff --git a/src/crypto/crypto_sig.cc b/src/crypto/crypto_sig.cc
@@ -215,7 +215,7 @@ ByteSource ConvertSignatureToDER(
 
   CHECK_NOT_NULL(data);
 
-  return ByteSource::Allocated(reinterpret_cast<char*>(data), len);
+  return ByteSource::Allocated(data, len);
 }
 
 void CheckThrow(Environment* env, SignBase::Error error) {

Original file line number	Diff line number	Diff line change
`@@ -215,7 +215,7 @@ ByteSource ConvertSignatureToDER(`
`215`	`215`
`216`	`216`	`CHECK_NOT_NULL(data);`
`217`	`217`
`218`		`- return ByteSource::Allocated(reinterpret_cast<char*>(data), len);`
	`218`	`+ return ByteSource::Allocated(data, len);`
`219`	`219`	`}`
`220`	`220`
`221`	`221`	`void CheckThrow(Environment* env, SignBase::Error error) {`