]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
secureserver.pl: fix stunnel version parsing
authorJay Satiro <raysatiro@yahoo.com>
Wed, 23 Aug 2023 06:49:47 +0000 (02:49 -0400)
committerJay Satiro <raysatiro@yahoo.com>
Mon, 28 Aug 2023 19:10:48 +0000 (15:10 -0400)
- Allow the stunnel minor-version version part to be zero.

Prior to this change with the stunnel version scheme of <major>.<minor>
if either part was 0 then version parsing would fail, causing
secureserver.pl to fail with error "No stunnel", causing tests that use
the SSL protocol to be skipped. As a practical matter this bug can only
be caused by a minor-version part of 0, since the major-version part is
always greater than 0.

Closes https://github.com/curl/curl/pull/11722

tests/secureserver.pl

index f94b410decf1fd8764b93474da0fa89c314947d9..1b71c36c7959c7dcb6fcf9ba6cdeec5747d67f96 100755 (executable)
@@ -218,7 +218,7 @@ foreach my $veropt (('-version', '-V')) {
     }
     last if($ver_major);
 }
-if((!$ver_major) || (!$ver_minor)) {
+if((!$ver_major) || !defined($ver_minor)) {
     if(-x "$stunnel" && ! -d "$stunnel") {
         print "$ssltext Unknown stunnel version\n";
     }