1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

thread_sync.c: pass proper argument to queue_sleep in rb_szqueue_push

Browse Source 
When I removed the SizeQueue#push timeout from my PR, I forgot to
update the `queue_sleep` parameters to be a `queue_sleep_arg`.

Somehow this worked on most archs, but on Solaris/Sparc it would
legitimately crash when trying to access the `timeout` and `end`
members of the struct.
This commit is contained in:
Jean Boussier 2022-08-04 10:37:46 +02:00
parent 00f411c58a
commit 6b2fc33ae2
Notes: git 2022-08-04 18:49:04 +09:00 
Merged: https://github.com/ruby/ruby/pull/6213
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
thread_sync.c
View File

@ -1277,7 +1277,13 @@ rb_szqueue_push(int argc, VALUE *argv, VALUE self)
ccan_list_add_tail(pushq, &queue_waiter.w.node);
sq->num_waiting_push++;
rb_ensure(queue_sleep, self, szqueue_sleep_done, (VALUE)&queue_waiter);
struct queue_sleep_arg queue_sleep_arg = {
.self = self,
.timeout = Qnil,
.end = 0
};
rb_ensure(queue_sleep, (VALUE)&queue_sleep_arg, szqueue_sleep_done, (VALUE)&queue_waiter);
}
}