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 1.01 years ago	Created:	2024-10-04 07:23:36 1.10 years ago
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.