Skip to content

Commit 9d583dd

Browse files
committed
merge revision(s) fba8aff, d8c6e91: [Backport #20592]
[Bug #20592] Fix segfault when sending NULL to freeaddrinfo On alpine freeaddrinfo does not accept NULL pointer Fix dangling `else`
1 parent 8a2e41d commit 9d583dd

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

ext/socket/raddrinfo.c

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -277,8 +277,9 @@ numeric_getaddrinfo(const char *node, const char *service,
277277
void
278278
rb_freeaddrinfo(struct rb_addrinfo *ai)
279279
{
280-
if (!ai->allocated_by_malloc)
281-
freeaddrinfo(ai->ai);
280+
if (!ai->allocated_by_malloc) {
281+
if (ai->ai) freeaddrinfo(ai->ai);
282+
}
282283
else {
283284
struct addrinfo *ai1, *ai2;
284285
ai1 = ai->ai;
@@ -423,7 +424,7 @@ do_getaddrinfo(void *ptr)
423424
arg->err = err;
424425
arg->gai_errno = gai_errno;
425426
if (arg->cancelled) {
426-
freeaddrinfo(arg->ai);
427+
if (arg->ai) freeaddrinfo(arg->ai);
427428
}
428429
else {
429430
arg->done = 1;

version.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
# define RUBY_VERSION_MINOR RUBY_API_VERSION_MINOR
1212
#define RUBY_VERSION_TEENY 3
1313
#define RUBY_RELEASE_DATE RUBY_RELEASE_YEAR_STR"-"RUBY_RELEASE_MONTH_STR"-"RUBY_RELEASE_DAY_STR
14-
#define RUBY_PATCHLEVEL 91
14+
#define RUBY_PATCHLEVEL 92
1515

1616
#include "ruby/version.h"
1717
#include "ruby/internal/abi.h"

0 commit comments

Comments
 (0)