First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 108471
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Alpha Porters <alpha@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Laurens Vets <laurens@daemon.be>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
soundtracker-0.6.7-r1.ebuild soundtracker-0.6.7-r1.ebuild text/plain Thomas Cort (RETIRED) 2006-01-12 12:23 0000 1.58 KB Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 108471 depends on: Show dependency tree
Bug 108471 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2005-10-08 03:26 0000 
Hi guys,

I've emerged Soundtracker 0.6.7 on alpha and it compiles and works.  I did not
run into bug 91619 or anything similar.

Best regards,
Laurens Vets

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

------- Comment #1 From Thomas Cort (RETIRED) 2006-01-12 12:23:39 0000 ------- 
Created an attachment (id=76935) [details]
soundtracker-0.6.7-r1.ebuild

I verified that the patch is correct. It removes a line in midi_init() in
app/midi-09x.c that free()'s a variable called "port_sub". Removing the line
doesn't cause a memory leak as the variable is allocated on the stack by the
line snd_seq_port_subscribe_alloca(&port_sub). free()'ing that memory caused
the crash. The crash doesn't happen all of the time because the line that
causes the error is in a block of code that only gets executed if
snd_seq_subscribe_port( midi_handle, port_sub) returns an error.

This ebuild applies the patch in attachement #67640. Download the patch to
soundtracker-0.6.7-invalid-free-fix.patch and put it in the files directory.

------- Comment #2 From Thomas Cort (RETIRED) 2006-01-12 12:27:07 0000 ------- 
Oops, that Attachment #76935 [details] was supposed to go on Bug #91618. It references
the patch in Attachment #67640 [details]. BTW, I tested the patch and soundtracker-0.6.7
and it builds and works on alpha.

------- Comment #3 From Thomas Cort (RETIRED) 2006-04-02 11:09:48 0000 ------- 
Added ~alpha. Thanks for testing this Laurens.
First Last Prev Next    No search results available      Search page      Enter new bug