From: Matt Corallo <649246+TheBlueMatt@users.noreply.github.com>
Date: Thu, 22 Jul 2021 14:17:09 +0000 (+0000)
Subject: Merge pull request #1008 from lightning-signer/2021-07-sync-no-std
X-Git-Tag: v0.0.100~24
X-Git-Url: http://git.bitcoin.ninja/index.cgi?p=rust-lightning;a=commitdiff_plain;h=41d8d4d6b277b7919ea847ffcfcaea59471ba4ff;hp=2e8f4fef2faa06a8752163bb6bc6908520632e27

Merge pull request #1008 from lightning-signer/2021-07-sync-no-std

Dummy sync implementation for no_std
---

diff --git a/lightning/src/util/message_signing.rs b/lightning/src/util/message_signing.rs
index 2055b408..8beff835 100644
--- a/lightning/src/util/message_signing.rs
+++ b/lightning/src/util/message_signing.rs
@@ -36,6 +36,11 @@ fn sigrec_encode(sig_rec: RecoverableSignature) -> Vec<u8> {
 }
 
 fn sigrec_decode(sig_rec: Vec<u8>) -> Result<RecoverableSignature, Error> {
+    // Signature must be 64 + 1 bytes long (compact signature + recovery id)
+    if sig_rec.len() != 65 {
+        return Err(Error::InvalidSignature);
+    }
+
     let rsig = &sig_rec[1..];
     let rid = sig_rec[0] as i32 - 31;