Change Details

Rationales from the PR description: - In CSignatureCache::ComputeEntry, change the way a vector pointer is resolved to prevent invoking undefined behavior if the vector is empty. - Sometimes alternative code without any raw data pointers is easier to read (refer to the respective commit message for details) - If the raw data pointer is needed, foo.data() should be preferred, as pointed out in the developer notes. This is a backport of core#21817

Rationales from the PR description: - In CSignatureCache::ComputeEntry, change the way a vector pointer is resolved to prevent invoking undefined behavior if the vector is empty. - Sometimes alternative code without any raw data pointers is easier to read (refer to the respective commit message for details) - If the raw data pointer is needed, foo.data() should be preferred, as pointed out in the developer notes. This is a backport of [[https://github.com/bitcoin/bitcoin/pull/21817 | core#21817]]

Rationales from the PR description: - In CSignatureCache::ComputeEntry, change the way a vector pointer is resolved to prevent invoking undefined behavior if the vector is empty. - Sometimes alternative code without any raw data pointers is easier to read (refer to the respective commit message for details) - If the raw data pointer is needed, foo.data() should be preferred, as pointed out in the developer notes. This is a backport of [[https://github.com/bitcoin/bitcoin/pull/21817 | core#21817]]