Bug 731

Summary: addseverity shouldn't strdup the string
Product: glibc Reporter: Roland McGrath <roland>
Component: libcAssignee: Roland McGrath <roland>
Status: RESOLVED FIXED    
Severity: normal CC: glibc-bugs
Priority: P2 Flags: fweimer: security-
Version: 2.3.4   
Target Milestone: ---   
Host: Target:
Build: Last reconfirmed:
Project(s) to access: ssh public key:
Bug Depends on:    
Bug Blocks: 724    

Description Roland McGrath 2005-02-16 04:35:09 UTC
This trunk change is needed on the 2.3 branch.

2005-01-27  Jakub Jelinek  <jakub@redhat.com>

	* stdlib/tst-fmtmsg.c: Include stdlib.h.

2005-01-25  Jakub Jelinek  <jakub@redhat.com>

	* stdlib/fmtmsg.c (addseverity): Remove new_string variable.
	(free_mem): Don't free string.
	* stdlib/tst-fmtmsg.c: Include string.h.
	(main): Add some more tests.

2005-01-14  Ulrich Drepper  <drepper@redhat.com>

	* stdlib/fmtmsg.c (internal_addseverity): Remove incorrect free call.
	* stdlib/tst-fmtmsg.c (main): Add another addseverity test.
Comment 1 Sourceware Commits 2005-02-16 10:37:21 UTC
Subject: Bug 731

CVSROOT:	/cvs/glibc
Module name:	libc
Branch: 	glibc-2_3-branch
Changes by:	roland@sources.redhat.com	2005-02-16 10:37:06

Modified files:
	stdlib         : tst-fmtmsg.c 

Log message:
	2005-01-27  Jakub Jelinek  <jakub@redhat.com>
	
	[BZ #731]
	* stdlib/tst-fmtmsg.c: Include stdlib.h.
	2005-01-25  Jakub Jelinek  <jakub@redhat.com>
	
	[BZ #731]
	* stdlib/fmtmsg.c (addseverity): Remove new_string variable.
	(free_mem): Don't free string.
	* stdlib/tst-fmtmsg.c: Include string.h.
	(main): Add some more tests.
	2005-01-14  Ulrich Drepper  <drepper@redhat.com>
	
	[BZ #731]
	* stdlib/fmtmsg.c (internal_addseverity): Remove incorrect free call.
	* stdlib/tst-fmtmsg.c (main): Add another addseverity test.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/libc/stdlib/tst-fmtmsg.c.diff?cvsroot=glibc&only_with_tag=glibc-2_3-branch&r1=1.1&r2=1.1.6.1

Comment 2 Sourceware Commits 2005-02-16 10:37:23 UTC
Subject: Bug 731

CVSROOT:	/cvs/glibc
Module name:	libc
Branch: 	glibc-2_3-branch
Changes by:	roland@sources.redhat.com	2005-02-16 10:37:11

Modified files:
	stdlib         : fmtmsg.c 

Log message:
	2005-01-25  Jakub Jelinek  <jakub@redhat.com>
	
	[BZ #731]
	* stdlib/fmtmsg.c (addseverity): Remove new_string variable.
	(free_mem): Don't free string.
	* stdlib/tst-fmtmsg.c: Include string.h.
	(main): Add some more tests.
	2005-01-14  Ulrich Drepper  <drepper@redhat.com>
	
	[BZ #731]
	* stdlib/fmtmsg.c (internal_addseverity): Remove incorrect free call.
	* stdlib/tst-fmtmsg.c (main): Add another addseverity test.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/libc/stdlib/fmtmsg.c.diff?cvsroot=glibc&only_with_tag=glibc-2_3-branch&r1=1.15&r2=1.15.4.1

Comment 3 Roland McGrath 2005-04-05 23:59:21 UTC
These changes are in the 2.3 branch as well as the trunk now.