Skip to content

Fix nightly psk build warnings#194

Merged
kaleb-himes merged 1 commit intowolfSSL:masterfrom
tmael:psk
Mar 20, 2020
Merged

Fix nightly psk build warnings#194
kaleb-himes merged 1 commit intowolfSSL:masterfrom
tmael:psk

Conversation

@tmael
Copy link
Copy Markdown
Contributor

@tmael tmael commented Mar 20, 2020

This PR fixes two build warnings listed below:

server-psk-nonblocking.c: In function ‘main’:

server-psk-nonblocking.c:260:29: warning: ‘select_ret’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                     else if (select_ret == TEST_TIMEOUT) {
                             ^
gcc -o server-tcp server-tcp.c -Wall -I/var/lib/jenkins/workspace/nightly-CDT-wolfssl-examples-repo-v-1-1/install-dir/include -Os -pthread -L/var/lib/jenkins/workspace/nightly-CDT-wolfssl-examples-repo-v-1-1/install-dir/lib -lm -lwolfssl
gcc -o server-psk server-psk.c -Wall -I/var/lib/jenkins/workspace/nightly-CDT-wolfssl-examples-repo-v-1-1/install-dir/include -Os -pthread -L/var/lib/jenkins/workspace/nightly-CDT-wolfssl-examples-repo-v-1-1/install-dir/lib -lm -lwolfssl
gcc -o client-psk-nonblocking client-psk-nonblocking.c -Wall -I/var/lib/jenkins/workspace/nightly-CDT-wolfssl-examples-repo-v-1-1/install-dir/include -Os -pthread -L/var/lib/jenkins/workspace/nightly-CDT-wolfssl-examples-repo-v-1-1/install-dir/lib -lm -lwolfssl
client-psk-nonblocking.c: In function ‘main’:

client-psk-nonblocking.c:207:17: warning: ‘select_ret’ may be used uninitialized in this function [-Wmaybe-uninitialized]
         else if (select_ret == TEST_TIMEOUT) {
                 ^

Copy link
Copy Markdown
Contributor

@kaleb-himes kaleb-himes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Tesfa!

@kaleb-himes kaleb-himes merged commit f6f8599 into wolfSSL:master Mar 20, 2020
yota22721 pushed a commit to yota22721/wolfssl-examples that referenced this pull request Jan 25, 2025
Fix nightly psk build warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants