fuzz: crypter: Abrt in __cxxabiv1::failed_throw #30251

1. 
   maflcko commented at 10:07 am on June 9, 2024: member

   From https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=69500

   In:

   • https://github.com/bitcoin/bitcoin/blob/4a020ca443ba370bf41583962d16aa8551876f53/src/wallet/test/fuzz/crypter.cpp#L65
   • https://github.com/bitcoin/bitcoin/blob/4a020ca443ba370bf41583962d16aa8551876f53/src/wallet/crypter.cpp#L98

   I haven’t looked into this in detail, but it may be allocating a large chunk of memory. Given that locked memory is only used for small chunks of memory (private keys and rng state), I wonder if it makes sense to fuzz inputs of this length?

2. maflcko added the label Tests on Jun 9, 2024
3. 
   brunoerg commented at 12:19 pm on June 10, 2024: contributor

   Maybe it’s not good to use ConsumeRandomLengthByteVector without limiting, worthes limiting its size. https://github.com/bitcoin/bitcoin/blob/cad127235e307d7de0e9cc04708dbd31aa6c24b0/src/wallet/test/fuzz/crypter.cpp#L59
4. fanquake closed this on Jul 15, 2024

   ---

5. fanquake referenced this in commit 35102d4928 on Jul 15, 2024

Contributors
maflcko brunoerg

Labels
Tests