Skip to content

initialize resp_length = 0 in tls.c#5897

Merged
dgarske merged 1 commit intowolfSSL:masterfrom
gojimmypi:gojimmypi-initialize-resp_length
Dec 16, 2022
Merged

initialize resp_length = 0 in tls.c#5897
dgarske merged 1 commit intowolfSSL:masterfrom
gojimmypi:gojimmypi-initialize-resp_length

Conversation

@gojimmypi
Copy link
Copy Markdown
Contributor

Description

This warning was being treated as an error:

Error    'resp_length' may be used uninitialized in this function: [..] wolfssl\src\tls.c

image

Fixes zd# n/a

Testing

How did you test?

Applied change and verified wolfcrypt/benchmark/benchmark and wolfcrypt/test/testwolfcrypt ran successfully.

git clone https://github.com/gojimmypi/wolfssl.git --branch gojimmypi-initialize-resp_length --depth 1
cd wolfssl
./autogen.sh
./configure
make all
./wolfcrypt/benchmark/benchmark
./wolfcrypt/test/testwolfcrypt

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@gojimmypi gojimmypi requested a review from dgarske December 15, 2022 22:16
@gojimmypi gojimmypi self-assigned this Dec 15, 2022
@dgarske dgarske assigned dgarske and unassigned gojimmypi Dec 15, 2022
@dgarske dgarske added the For This Release Release version 5.9.1 label Dec 16, 2022
@dgarske dgarske merged commit 502fd84 into wolfSSL:master Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For This Release Release version 5.9.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants