* util.c (ruby_strdup): remove unnecessary code. (xmalloc never

  returns NULL.)

* util.c (ruby_getcwd): fix memory leak on failure.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_8@6886 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 5 insertions, 2 deletions

diff --git a/util.c b/util.c
index c29433937e..6f76519176 100644
--- a/util.c
+++ b/util.c
@@ -629,7 +629,6 @@ ruby_strdup(str)
     int len = strlen(str) + 1;
 
     tmp = xmalloc(len);
-    if (tmp == NULL) return NULL;
     memcpy(tmp, str, len);
 
     return tmp;
@@ -643,7 +642,10 @@ ruby_getcwd()
     char *buf = xmalloc(size);
 
     while (!getcwd(buf, size)) {
-	if (errno != ERANGE) rb_sys_fail("getcwd");
+	if (errno != ERANGE) {
+	    free(buf);
+	    rb_sys_fail("getcwd");
+	}
 	size *= 2;
 	buf = xrealloc(buf, size);
     }
@@ -654,6 +656,7 @@ ruby_getcwd()
     char *buf = xmalloc(PATH_MAX+1);
 
     if (!getwd(buf)) {
+	free(buf);
 	rb_sys_fail("getwd");
     }
 #endif