bug#9815: [bug #25525] Segfault using goops

Update of bug #25525 (project guile): Status:None => Fixed Open/Closed:Open => Closed ___ Follow-up Comment #8: Thank you for the fix

[bug #25525] Segfault using goops

Follow-up Comment #7, bug #25525 (project guile): added goops4.diff, a git diff that fixes c stylish errors and use scm_is_eq in stead of a direct comparision. /Stefan (file #23943) ___ Additional Item Attachment: File name: goops4.diff

Re: [bug #25525] Segfault using goops

Hi Stefan, I appreciate very much your work on the bugs! Your fixes have been thoughtful and mostly correct, it seems to me. In this case, your approach sounds about right. There are a number of style errors that need fixing, though: diff --git a/libguile/goops.c b/libguile/goops.c ind

[bug #25525] Segfault using goops

Follow-up Comment #6, bug #25525 (project guile): I recognized that the layout problem appears as well if you inherit from first and then a class which does not contain I did fix this. By postponing the addition of variables to the last in order for them to always get the correct index in the

[bug #25525] Segfault using goops

Follow-up Comment #5, bug #25525 (project guile): Ok, I did an attempt to make an acceptable fix to this. The problem was that in the name slot order of the layout changed due to the new slot again called name. This cassed lookup code that assumes a fixed layout to fetch the wrong values due to

[bug #25525] Segfault using goops

Follow-up Comment #4, bug #25525 (project guile): This bug is still present in git. ___ Reply to this item at: ___ Message sent via/by Savannah http:

[bug #25525] Segfault using goops

Follow-up Comment #3, bug #25525 (project guile): a cut and paste bug, the function is build_slots_list in goops.c Anyhow I saw that I can enable a pair test, with a debug option in the compilation. So, as always, people are smart. ___ R

[bug #25525] Segfault using goops

Follow-up Comment #2, bug #25525 (project guile): The reason is that code assume cpl to be a null terminated list but it's not due to the bug, then a cdr will make the crash. Now this is a symptom and not the cause. But we can avoid a segmentation fault by making sure that we check for a pair al

[bug #25525] Segfault using goops

Follow-up Comment #1, bug #25525 (project guile): In case it helps until this is fixed properly... This appears to be specific to the combination of - the use of "name" in the definition of - the use of as a superclass. I would guess related to the fact that already has a slot called "name".

[bug #25525] Segfault using goops

URL: Summary: Segfault using goops Project: Guile Submitted by: burschik Submitted on: Mon 09 Feb 2009 10:25:50 AM EET Category: None Severity: 3 - Normal