FAB18529 added nil check in channel header parsing

Fuzz testing has reported SEGV while sending incomplete/null message request to orderer. Signed-off-by: Parameswaran Selvam <[email protected]> (cherry picked from commit 540fff8)
hyperledger · Aug 19, 2021 · befdaee · befdaee
1 parent c1b298d
commit befdaee
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/protoutil/commonutils.go b/protoutil/commonutils.go
@@ -212,6 +212,10 @@ func IsConfigBlock(block *cb.Block) bool {
 
 // ChannelHeader returns the *cb.ChannelHeader for a given *cb.Envelope.
 func ChannelHeader(env *cb.Envelope) (*cb.ChannelHeader, error) {
+	if env == nil {
+		return nil, errors.New("Invalid envelope payload. can't be nil")
+	}
+
 	envPayload, err := UnmarshalPayload(env.Payload)
 	if err != nil {
 		return nil, err