wireguard: dont stacksmash bad peer base64 keys 25/35625/2
authorJon Loeliger <jdl@netgate.com>
Thu, 10 Mar 2022 17:23:27 +0000 (11:23 -0600)
committerMatthew Smith <mgsmith@netgate.com>
Mon, 21 Mar 2022 16:25:21 +0000 (16:25 +0000)
commitd23c55fa56a0f564be290f9f301094fd5ba45361
tree338a4b3c20a20c05ce231baafd742ea7d39573da
parentf47917959872afe2aff464343944bbffee545308
wireguard: dont stacksmash bad peer base64 keys

Just like commit 252647482b24bb3474e8f13bc86100718176832f
did for Wireguard interface keys, prevent stack smashing
the peer keys.

Integer math on 32 bytes of base64 data might yield 33 bytes
of data in some poorly formed user input of private key values.
Rather than smashing the stack (detected) and aborting, simply
allow for the possible yet irrelevant 33-rd byte of data.

Type: fix
Fixes: edca1325cf296bd0f5ff422fc12de2ce7a7bad88
Signed-off-by: Jon Loeliger <jdl@netgate.com>
Change-Id: I9f77b3faaaa01d3123b356c958db60c87238db9c
src/plugins/wireguard/wireguard_cli.c