github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
nwc10 good *, #moarvm 07:09
japhb good *! 07:11
nine Oh... SUMMARY: AddressSanitizer: heap-buffer-overflow src/spesh/stats.c:40 in incomplete_type_tuple 12:26
0x62f000000040 is located 960 bytes to the left of 51200-byte region [0x62f000000400,0x62f00000cc00)
allocated by Thread 0 in #2 0x7f78fdb0737b in setup_bin src/gc/gen2.c:27 12:57
nine Is it me, or is MVM_spesh_sim_stack_gc_mark neglecting to mark the MVMSpeshSimStackFrame's arg_types's type and decont_type? 13:12
nwc10 this is one of those: there are many things where I think "[...] this is out of my league" 13:14
so, er, sorry, can't help.
write the question on a beer bottle and post it to Prague?
dogbert11 nine: so which bug are you hunting now? 13:21
nine Well actually I'm continuing my work on getting native call memory management right once and for all. That heap-buffer-overflow was just accidental. Of course it's not reproducible, but the bug looks pretty clear from reading the code. 13:33
I fully understand what's going on and why it's buggy. It just seems _too_ clear to me. How could this have hidden in plain sight so well so far? Makes me wonder if I miss something. 13:34
dogbert11 so the investigation continues 13:36
lizmat nine: maybe it is *not* hiding, just responsible for many one-time event crashes ? 13:37
nine lizmat: I've had that hope too often lately, so I'm wary 13:38
dogbert11 is it a specific bug report that has turned your attention towards NativeCall? 13:41
nine No, it's actually the continuation of my ASAN fixes that I did at the start of the year. The remaining failures in make test are all because of NativeCall missing the required API for telling the VM what to free and what to leave to the application. 13:44
I've already come pretty far with that, but then got side tracked with a few more urgent bug reports 13:45
dogbert11 Aha, let me know if you need any help in reproing something 13:55
Geth MoarVM/fix_missing_gc_mark_in_spesh_sim: 14d9dd566a | (Stefan Seifert)++ | src/spesh/stats.c
Fix missing gc_mark of simstackframe's arg_types

Found through: AddressSanitizer: heap-buffer-overflow src/spesh/stats.c:40 in incomplete_type_tuple x62f000000040 is located 960 bytes to the left of 51200- byte region [0x62f000000400,0x62f00000cc00) allocated by thread 0 in setup_bin src/gc/gen2.c:27
14:03
MoarVM: niner++ created pull request #1478:
Fix missing gc_mark of simstackframe's arg_types
dogbert11 nine: btw, it seems as if Dyncall 1.2 was released in January 14:08
sena_kun remembers last time it was updated and shudders 14:09
nine Darn....I just got that heap-buffer-overflow again. But my supposed fix was already in place :/ 14:10
dogbert11 oops 14:10
nine The good new is: it actually seems reproducible now 14:11
dogbert11 That is indeed good news
dogbert11 dyncall.org/changelog 14:17
lizmat "support for loading macos >= 11.0.1 (Big Sur) dylibs provided by system's "built-in dynamic linker cache of all system-provided libraries" (those dylibs are no longer present on the fs)" 14:39
wonder if that's the reason for these strange failures I see for IRC::Client on MacOS
oth, that is on 10.x 14:40
*otoh
Geth MoarVM/2021.04-release: ae63690579 | Altai-man++ | docs/ChangeLog
Update ChangeLog for 2021.04 release
15:20
MoarVM: Altai-man++ created pull request #1479:
Update ChangeLog for 2021.04 release
15:21
sena_kun I'll cut a release once ^ is merged.
dogbert11 ++sena_kun 15:24
sena_kun anyone wants to check its sanity? :) 15:27
Geth MoarVM: ae63690579 | Altai-man++ | docs/ChangeLog
Update ChangeLog for 2021.04 release
15:43
MoarVM: 90f34e0db6 | Altai-man++ (committed using GitHub Web editor) | docs/ChangeLog
Merge pull request #1479 from MoarVM/2021.04-release

Update ChangeLog for 2021.04 release
MoarVM/2021.04: 239d0b7c90 | Altai-man++ | VERSION
Bump VERSION for release
15:50
MoarVM: Altai-man++ created pull request #1480:
2021.04 release
15:51
MoarVM: 239d0b7c90 | Altai-man++ | VERSION
Bump VERSION for release
15:52
MoarVM: ce733ddf86 | Altai-man++ (committed using GitHub Web editor) | VERSION
Merge pull request #1480 from MoarVM/2021.04

2021.04 release