severity 302200 normal
tags 302200 +pending +patch
thanks

On Wed, Mar 30, 2005 at 06:06:28PM +0200, Michael Vistein wrote:
> Package: e2fsprogs
> Version: 1.37-1
> Severity: grave
> Justification: renders package unusable

This is a problem only on IA64, and only breaks mke2fs, and not the
other e2fsprogs programs.  The problem seems to be the malloc() goes
wonky if we don't include stdlib.h.  The following patch fixes the
problem.  I'll upload a fixed set of sources very shortly.

                                                        - Ted

# This is a BitKeeper generated diff -Nru style patch.
#
# ChangeSet
#   2005/03/31 00:01:33-05:00 [EMAIL PROTECTED] 
#   ostype.c (e2p_os2string): Check to make sure malloc() is
#       successful before attempting to copy into it.  Add
#       #include of stdlib.h to fix a core dump bug on the IA64
#       architecture.  (Addresses Debian Bug #302200)
# 
# lib/e2p/ostype.c
#   2005/03/31 00:01:33-05:00 [EMAIL PROTECTED] +3 -1
#   ostype.c (e2p_os2string): Check to make sure malloc() is
#       successful before attempting to copy into it.  Add
#       #include of stdlib.h to fix a core dump bug on the IA64
#       architecture.  (Addresses Debian Bug #302200)
# 
# lib/e2p/ChangeLog
#   2005/03/31 00:01:33-05:00 [EMAIL PROTECTED] +7 -0
#   Update log
# 
diff -Nru a/lib/e2p/ChangeLog b/lib/e2p/ChangeLog
--- a/lib/e2p/ChangeLog 2005-03-31 00:01:46 -05:00
+++ b/lib/e2p/ChangeLog 2005-03-31 00:01:46 -05:00
@@ -1,3 +1,10 @@
+2005-03-30  Theodore Ts'o  <[EMAIL PROTECTED]>
+
+       * ostype.c (e2p_os2string): Check to make sure malloc() is
+               successful before attempting to copy into it.  Add
+               #include of stdlib.h to fix a core dump bug on the IA64
+               architecture.  (Addresses Debian Bug #302200)
+
 2005-03-21  Theodore Ts'o  <[EMAIL PROTECTED]>
 
        * Release of E2fsprogs 1.37
diff -Nru a/lib/e2p/ostype.c b/lib/e2p/ostype.c
--- a/lib/e2p/ostype.c  2005-03-31 00:01:46 -05:00
+++ b/lib/e2p/ostype.c  2005-03-31 00:01:46 -05:00
@@ -9,6 +9,7 @@
 
 #include "e2p.h"
 #include <string.h>
+#include <stdlib.h>
 
 const char *os_tab[] =
        { "Linux", 
@@ -32,7 +33,8 @@
                os = "(unknown os)";
 
         ret = malloc(strlen(os)+1);
-        strcpy(ret, os);
+       if (ret)
+               strcpy(ret, os);
         return ret;
 }
 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to