TclTLS: View Ticket

2024-11-04
02:46		• Closed ticket [fee1ffa1b6]: Error in mode checking in latest checkin 1505883e4a from 2024-07-01 plus 3 other changes artifact: d7b9f76ceb user: bohagan
2024-10-28
03:04		• Fixed ticket [fee1ffa1b6]. artifact: 04f7923dd1 user: bohagan
2024-10-04
09:42		• Ticket [fee1ffa1b6]: 6 changes artifact: 73f193cb73 user: anonymous
07:23		• New ticket [fee1ffa1b6]. artifact: 47ac8f1513 user: anonymous

Ticket Hash:	fee1ffa1b6037cb15b592ad550d21760e1548eda
Title:	Error in mode checking in latest checkin 1505883e4a from 2024-07-01
Status:	Closed	Type:	Code Defect
Severity:	Severe	Priority:	Immediate
Subsystem:		Resolution:	Fixed
Last Modified:	2024-11-04 02:46:49
Version Found In:	1505883e4a from 2024-07-01
User Comments:
anonymous added on 2024-10-04 07:23:36: Compiling tls.c on macoS (17.0/arm) yields the following warnings which I assume are errors: /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:1769:18: warning: variable 'abort' set but not used [-Wunused-but-set-variable] 1769 \| int off = 0, abort = 0; \| ^ /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2222:11: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand] 2222 \| if (mode && SSL_VERIFY_FAIL_IF_NO_PEER_CERT) { \| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2222:11: note: use '&' for a bitwise operation 2222 \| if (mode && SSL_VERIFY_FAIL_IF_NO_PEER_CERT) { \| ^~ \| & /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2222:11: note: remove constant to silence this warning 2222 \| if (mode && SSL_VERIFY_FAIL_IF_NO_PEER_CERT) { \| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2225:11: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand] 2225 \| if (mode && SSL_VERIFY_CLIENT_ONCE) { \| ^ ~~~~~~~~~~~~~~~~~~~~~~ /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2225:11: note: use '&' for a bitwise operation 2225 \| if (mode && SSL_VERIFY_CLIENT_ONCE) { \| ^~ \| & /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2225:11: note: remove constant to silence this warning 2225 \| if (mode && SSL_VERIFY_CLIENT_ONCE) { \| ^~~~~~~~~~~~~~~~~~~~~~~~~ /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2228:11: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand] 2228 \| if (mode && SSL_VERIFY_POST_HANDSHAKE) { \| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~ /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2228:11: note: use '&' for a bitwise operation 2228 \| if (mode && SSL_VERIFY_POST_HANDSHAKE) { \| ^~ \| & /opt/projects/prs/g481/src/tcltls-1505883e4a/generic/tls.c:2228:11: note: remove constant to silence this warning 2228 \| if (mode && SSL_VERIFY_POST_HANDSHAKE) { \| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ 4 warnings generated. Fix: Change the '&&' into '&' since the openssl function returns an int containing a bitmask. I am not converned about the unused variable `abort`. I applied this modificationlocally and everything seems to work correctly. anonymous added on 2024-10-04 09:42:07: Corrected typo in description. bohagan added on 2024-10-28 03:04:17: This should be corrected in [c747afd2002a4cbf]. Let me know if you agree.