You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2021/10/27 14:18:29 UTC

[GitHub] [trafficserver] traeak opened a new pull request #8462: ssl_verify_test: clang-analyzer fix to account for null sni_name

traeak opened a new pull request #8462:
URL: https://github.com/apache/trafficserver/pull/8462


   clang-analyzer was upset due to a debug statement using a char* that could potentially be null.  This should resolve this error.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] zwoop commented on pull request #8462: ssl_verify_test: clang-analyzer fix to account for null sni_name

Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #8462:
URL: https://github.com/apache/trafficserver/pull/8462#issuecomment-972337714


   Cherry-picked to v9.2.x


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] traeak merged pull request #8462: ssl_verify_test: clang-analyzer fix to account for null sni_name

Posted by GitBox <gi...@apache.org>.
traeak merged pull request #8462:
URL: https://github.com/apache/trafficserver/pull/8462


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] zwoop commented on pull request #8462: ssl_verify_test: clang-analyzer fix to account for null sni_name

Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #8462:
URL: https://github.com/apache/trafficserver/pull/8462#issuecomment-972338788


   Thanks for the fix, please remember to mark for the appropriate release branches (10-Dev is not relevant here, since this is not a compatibility issue)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [trafficserver] randall commented on pull request #8462: ssl_verify_test: clang-analyzer fix to account for null sni_name

Posted by GitBox <gi...@apache.org>.
randall commented on pull request #8462:
URL: https://github.com/apache/trafficserver/pull/8462#issuecomment-953055310


   [approve ci autest]


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org