Skip to content

Commit 77349f4

Browse files
f - Match Test vector packet
1 parent 0662e2a commit 77349f4

File tree

2 files changed

+23
-12
lines changed

2 files changed

+23
-12
lines changed

lightning/src/ln/onion_route_tests.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,7 @@ fn test_onion_failure() {
344344
// break the first (non-final) hop payload by swapping the realm (0) byte for a byte
345345
// describing a length-1 TLV payload, which is obviously bogus.
346346
new_payloads[0].data[0] = 1;
347-
msg.onion_routing_packet = onion_utils::construct_onion_packet_bogus_hopdata(new_payloads, onion_keys, [0; 32], &payment_hash);
347+
msg.onion_routing_packet = onion_utils::construct_onion_packet_with_writable_hopdata(new_payloads, onion_keys, [0; 32], &payment_hash);
348348
}, ||{}, true, Some(PERM|22), Some(NetworkUpdate::ChannelFailure{short_channel_id, is_permanent: true}), Some(short_channel_id));
349349

350350
// final node failure
@@ -361,7 +361,7 @@ fn test_onion_failure() {
361361
// break the last-hop payload by swapping the realm (0) byte for a byte describing a
362362
// length-1 TLV payload, which is obviously bogus.
363363
new_payloads[1].data[0] = 1;
364-
msg.onion_routing_packet = onion_utils::construct_onion_packet_bogus_hopdata(new_payloads, onion_keys, [0; 32], &payment_hash);
364+
msg.onion_routing_packet = onion_utils::construct_onion_packet_with_writable_hopdata(new_payloads, onion_keys, [0; 32], &payment_hash);
365365
}, ||{}, false, Some(PERM|22), Some(NetworkUpdate::ChannelFailure{short_channel_id, is_permanent: true}), Some(short_channel_id));
366366

367367
// the following three with run_onion_failure_test_with_fail_intercept() test only the origin node

lightning/src/ln/onion_utils.rs

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,15 @@ pub(super) fn gen_ammag_from_shared_secret(shared_secret: &[u8]) -> [u8; 32] {
8282
Hmac::from_engine(hmac).into_inner()
8383
}
8484

85+
#[inline]
86+
#[allow(dead_code)]
87+
pub(super) fn gen_pad_from_shared_secret(shared_secret: &[u8]) -> [u8; 32] {
88+
assert_eq!(shared_secret.len(), 32);
89+
let mut hmac = HmacEngine::<Sha256>::new(&[0x70, 0x61, 0x64]); // pad
90+
hmac.input(&shared_secret);
91+
Hmac::from_engine(hmac).into_inner()
92+
}
93+
8594
pub(crate) fn next_hop_packet_pubkey<T: secp256k1::Signing + secp256k1::Verification>(secp_ctx: &Secp256k1<T>, packet_pubkey: PublicKey, packet_shared_secret: &[u8; 32]) -> Result<PublicKey, secp256k1::Error> {
8695
let blinding_factor = {
8796
let mut sha = Sha256::engine();
@@ -224,9 +233,9 @@ pub(super) fn construct_onion_packet(payloads: Vec<msgs::OnionHopData>, onion_ke
224233
}
225234

226235
#[cfg(test)]
227-
// Used in testing to write bogus OnionHopDatas, which is otherwise not representable in
228-
// msgs::OnionHopData.
229-
pub(super) fn construct_onion_packet_bogus_hopdata<HD: Writeable>(payloads: Vec<HD>, onion_keys: Vec<OnionKeys>, prng_seed: [u8; 32], associated_data: &PaymentHash) -> msgs::OnionPacket {
236+
/// Used in testing to write bogus `BogusOnionHopData` as well as `RawOnionHopData`, which is
237+
/// otherwise not representable in `msgs::OnionHopData`.
238+
pub(super) fn construct_onion_packet_with_writable_hopdata<HD: Writeable>(payloads: Vec<HD>, onion_keys: Vec<OnionKeys>, prng_seed: [u8; 32], associated_data: &PaymentHash) -> msgs::OnionPacket {
230239
let mut packet_data = [0; ONION_DATA_LEN];
231240

232241
let mut chacha = ChaCha20::new(&prng_seed, &[0; 8]);
@@ -765,6 +774,10 @@ mod tests {
765774

766775
use super::OnionKeys;
767776

777+
fn get_test_session_key() -> SecretKey {
778+
SecretKey::from_slice(&hex::decode("4141414141414141414141414141414141414141414141414141414141414141").unwrap()[..]).unwrap()
779+
}
780+
768781
fn build_test_onion_keys() -> Vec<OnionKeys> {
769782
// Keys from BOLT 4, used in both test vector tests
770783
let secp_ctx = Secp256k1::new();
@@ -800,9 +813,7 @@ mod tests {
800813
payment_params: None,
801814
};
802815

803-
let session_priv = SecretKey::from_slice(&hex::decode("4141414141414141414141414141414141414141414141414141414141414141").unwrap()[..]).unwrap();
804-
805-
let onion_keys = super::construct_onion_keys(&secp_ctx, &route.paths[0], &session_priv).unwrap();
816+
let onion_keys = super::construct_onion_keys(&secp_ctx, &route.paths[0], &get_test_session_key()).unwrap();
806817
assert_eq!(onion_keys.len(), route.paths[0].len());
807818
onion_keys
808819
}
@@ -927,11 +938,11 @@ mod tests {
927938
let expected_serialized_hop_4_payload = &hex::decode("1202022710040203e806080000000000000004").unwrap()[..];
928939
assert_eq!(hop_4_serialized_payload, expected_serialized_hop_4_payload);
929940

930-
let packet: msgs::OnionPacket = super::construct_onion_packet_with_init_noise::<_, _>(payloads, onion_keys, super::FixedSizeOnionPacket([0; super::ONION_DATA_LEN]), Some(&PaymentHash([0x42; 32])));
941+
let pad_keytype_seed = super::gen_pad_from_shared_secret(&get_test_session_key().secret_bytes());
942+
943+
let packet: msgs::OnionPacket = super::construct_onion_packet_with_writable_hopdata::<_>(payloads, onion_keys, pad_keytype_seed, &PaymentHash([0x42; 32]));
931944

932-
// Note that the resulting onion differs from the test vectors due to our
933-
// `construct_onion_packet_with_init_noise` implementation.
934-
assert_eq!(packet.encode(), hex::decode("0002EEC7245D6B7D2CCB30380BFBE2A3648CD7A942653F5AA340EDCEA1F283686619F7F3416A5AA36DC7EEB3EC6D421E9615471AB84C75A1B96D6BDE2F3BC5450EFB8B10E7EE7D90585AAF084E13A66E3B2BD0E926CCD8D263072206DBED0234F6505E21E282ABD8C0E4F5B9FF8042800BBAB065036EADD0149B37F27DDE664725A49866E052E809D2B0198AB9610FAA656BBF4EC516763A59F8F42C171B179166BA38958D4F51B3E73E6C7DD77BA8F874C673E50D1E7C243185154C2FC9B3EEB2F6D1EE1FA4F9D21A163EDF0FB445D5C8D7D688D6DD9C541762BF922EA0F2E798894B55B46EB286551DEFFD0D7506DFE7842FFA1E3C8A34449305546275AFB233ACF4323974680779F1A964CFE65EC142A1495A86666F439AD1D11EA1BB52A2B349A71C778F911ED23D39080D6B9D207A503EE6F6299C92D6361BC335FCBF9B5CD44747AADCE2CE6069CFDC3D671DAEF9F8AE590CF93D957C9E873E9A1BC62D9640DC8FC39C14902D49A1C80239B6C5B7FD91D05878CBF5FFC7DB2569F47C43D6C0D27C438ABFF276E87364DEB8858A37E5A62C446AF95D8B786EAF0B5FCF78D98B41496794F8DCAAC4EEF34B2ACFB94C7E8C32A9E9866A8FA0B6F2A06F00A1CCDE569F97EEC05C803BA7500ACC96691D8898D73D8E6A47B8F43C3D5DE74458D20EDA61474C426359677001FBD75A74D7D5DB6CB4FEB83122F133206203E4E2D293F838BF8C8B3A29ACB321315100B87E80E0EDB272EE80FDA944E3FB6084ED4D7F7C7D21C69D9DA43D31A90B70693F9B0CC3EAC74C11AB8FF655905688916CFA4EF0BD04135F2E50B7C689A21D04E8E981E74C6058188B9B1F9DFC3EEC6838E98809535C2244B757343CFC5E2AAD8CBF22AA2C4A268591974BD98206A5B508BE5E3CA8F956BDC346A51898153F18173F1C77DACFF1FD78B790E253D3C998AA22404331F340AF9567A8A398A1155B6A2754B8C7817BE793A96FB04A9F63D9963B82ECA10272B1357D85340715B0FDAE9C68D89FAAD20547C82432B1F6F7584F3DFA39F6DC83FE17351C1D15F7D8A13B64DA9CC2C7CCE74A5AAAE9C58FEB9947258416DFF072C34F7ECCA923701CD59CEC5F6B8AEFBCBED564EF78C84277A1F8C09AED9A1764EFB8BF6FCE557C4030BD65DF7576B00A8F22ABA138E9441F7EB133724E9A95AC6581E1E48992714FC1DB3C34B05BB7AF5BC2E1DEABC069F1E70B5E60943E2F4EC01D8B5F2DEE087879832EEC4D5060FA08B81B786DB3B4B10693CE3368442C8CAF17FB156182967986EF4171E089069142B8332AD26B67BFA48DD1752B594A408D686151AE0C8A1B12E5D76493E2A6A5C7411716AB10D1075F28941C6E0FD68282A1F0737273D8E704C9975CF99B8D2453B88C2C6DF29019FFDD80688EDCABF4C4D41F24F6790D7065A303AD171483184854A699886D610289D2D9FE68D56E09ACCAD90FF3D65783420662FF3F312F9E2B7E8F5C425E3B37D675556BD710EA2ABDDF24F6C9D291235F622827E0BE4843258BA014B45C2D520A277AB9EFCA564D3CA25D50BF8958CA51C0FBEB9B61C0454E582005505E1A63AEBDB9F0C1E63F1EB39EBCC69F57E9A3030B47FD9754DDA90EBF701288FC982418D5CD36049BC166EC3E808CEA2895243EFD9A7657C58CB33C93DADC75C06E211201EBE43B8BDA0F4BFF5B39DE232DAA33D6FEF9A1125149067B00C2DF4D53549CD6330A2307C4ADD4F11127A677F1C9ED24A7B9A48848A561BB05001D5736994975E5324CB02E46169D9598E5EF442DF21509746B50D1B971A4E53190BEDC858F7DFA6D656016F0D026FF49A874327329356551D504EBBC9C47A441D427247DF9896177C020F3CFC5D6570A292E3D492BF494F557B29EE6A92BB4EAE7E5B3BAFABD9845E984130F0327FB38A8067BAFADF5744FAA446C16").unwrap());
945+
assert_eq!(packet.encode(), hex::decode("0002EEC7245D6B7D2CCB30380BFBE2A3648CD7A942653F5AA340EDCEA1F283686619F7F3416A5AA36DC7EEB3EC6D421E9615471AB870A33AC07FA5D5A51DF0A8823AABE3FEA3F90D387529D4F72837F9E687230371CCD8D263072206DBED0234F6505E21E282ABD8C0E4F5B9FF8042800BBAB065036EADD0149B37F27DDE664725A49866E052E809D2B0198AB9610FAA656BBF4EC516763A59F8F42C171B179166BA38958D4F51B39B3E98706E2D14A2DAFD6A5DF808093ABFCA5AEAACA16EDED5DB7D21FB0294DD1A163EDF0FB445D5C8D7D688D6DD9C541762BF5A5123BF9939D957FE648416E88F1B0928BFA034982B22548E1A4D922690EECF546275AFB233ACF4323974680779F1A964CFE687456035CC0FBA8A5428430B390F0057B6D1FE9A8875BFA89693EEB838CE59F09D207A503EE6F6299C92D6361BC335FCBF9B5CD44747AADCE2CE6069CFDC3D671DAEF9F8AE590CF93D957C9E873E9A1BC62D9640DC8FC39C14902D49A1C80239B6C5B7FD91D05878CBF5FFC7DB2569F47C43D6C0D27C438ABFF276E87364DEB8858A37E5A62C446AF95D8B786EAF0B5FCF78D98B41496794F8DCAAC4EEF34B2ACFB94C7E8C32A9E9866A8FA0B6F2A06F00A1CCDE569F97EEC05C803BA7500ACC96691D8898D73D8E6A47B8F43C3D5DE74458D20EDA61474C426359677001FBD75A74D7D5DB6CB4FEB83122F133206203E4E2D293F838BF8C8B3A29ACB321315100B87E80E0EDB272EE80FDA944E3FB6084ED4D7F7C7D21C69D9DA43D31A90B70693F9B0CC3EAC74C11AB8FF655905688916CFA4EF0BD04135F2E50B7C689A21D04E8E981E74C6058188B9B1F9DFC3EEC6838E9FFBCF22CE738D8A177C19318DFFEF090CEE67E12DE1A3E2A39F61247547BA5257489CBC11D7D91ED34617FCC42F7A9DA2E3CF31A94A210A1018143173913C38F60E62B24BF0D7518F38B5BAB3E6A1F8AEB35E31D6442C8ABB5178EFC892D2E787D79C6AD9E2FC271792983FA9955AC4D1D84A36C024071BC6E431B625519D556AF38185601F70E29035EA6A09C8B676C9D88CF7E05E0F17098B584C4168735940263F940033A220F40BE4C85344128B14BEB9E75696DB37014107801A59B13E89CD9D2258C169D523BE6D31552C44C82FF4BB18EC9F099F3BF0E5B1BB2BA9A87D7E26F98D294927B600B5529C47E04D98956677CBCEE8FA2B60F49776D8B8C367465B7C626DA53700684FB6C918EAD0EAB8360E4F60EDD25B4F43816A75ECF70F909301825B512469F8389D79402311D8AECB7B3EF8599E79485A4388D87744D899F7C47EE644361E17040A7958C8911BE6F463AB6A9B2AFACD688EC55EF517B38F1339EFC54487232798BB25522FF4572FF68567FE830F92F7B8113EFCE3E98C3FFFBAEDCE4FD8B50E41DA97C0C08E423A72689CC68E68F752A5E3A9003E64E35C957CA2E1C48BB6F64B05F56B70B575AD2F278D57850A7AD568C24A4D32A3D74B29F03DC125488BC7C637DA582357F40B0A52D16B3B40BB2C2315D03360BC24209E20972C200566BCF3BBE5C5B0AEDD83132A8A4D5B4242BA370B6D67D9B67EB01052D132C7866B9CB502E44796D9D356E4E3CB47CC527322CD24976FE7C9257A2864151A38E568EF7A79F10D6EF27CC04CE382347A2488B1F404FDBF407FE1CA1C9D0D5649E34800E25E18951C98CAE9F43555EEF65FEE1EA8F15828807366C3B612CD5753BF9FB8FCED08855F742CDDD6F765F74254F03186683D646E6F09AC2805586C7CF11998357CAFC5DF3F285329366F475130C928B2DCEBA4AA383758E7A9D20705C4BB9DB619E2992F608A1BA65DB254BB389468741D0502E2588AEB54390AC600C19AF5C8E61383FC1BEBE0029E4474051E4EF908828DB9CCA13277EF65DB3FD47CCC2179126AAEFB627719F421E20").unwrap());
935946
}
936947

937948
#[test]

0 commit comments

Comments
 (0)