Bug 24284 – Skin changing causes SIGSEGV

Bug 24284 - Skin changing causes SIGSEGV

Summary:	Skin changing causes SIGSEGV

Status:	CLOSED RAWHIDE

Aliases:	None

Product:	Red Hat Raw Hide
Classification:	Retired
Component:	xmms (Show other bugs)
Sub Component:
Version:	1.0
Hardware:	i386 Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Bill Nottingham
QA Contact:
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2001-01-18 08:39 EST by Stefan Sorensen
Modified:	2014-03-16 22:18 EDT (History)
CC List:	1 user (show)

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2001-01-18 08:40:44 EST
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Patch to fix (should be integrated into tmpdir patch) (245 bytes, patch) 2001-01-18 08:40 EST, Stefan Sorensen	no flags	Details \| Diff
Replacement for tmpdir patch. (1.41 KB, patch) 2001-01-18 17:20 EST, Stefan Sorensen	no flags	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Stefan Sorensen 2001-01-18 08:39:27 EST

The tmpdir fix included in xmms-1.2.4-5 breaks skin changing. The patch
moves the tmpdir var to the back, but doesn't remove the free call.

Comment 1 Stefan Sorensen 2001-01-18 08:40:42 EST

Created attachment 7789 [details]
Patch to fix (should be integrated into tmpdir patch)

Comment 2 Bill Nottingham 2001-01-18 13:28:52 EST

*slaps self on forehead*

Duh, that makes sense.

Fixed in 1.2.4-6; thanks for the report.

Comment 3 Stefan Sorensen 2001-01-18 17:19:52 EST

Well, the fix wasn't quite enough, xmms would save the tmpdir as the
path to the skin. the easy fix is to create the tmpdir in the directory
returned by g_get_tmp_dir().

Comment 4 Stefan Sorensen 2001-01-18 17:20:34 EST

Created attachment 7810 [details]
Replacement for tmpdir patch.

Note You need to log in before you can comment on or make changes to this bug.