Changeset 157 in libcf for trunk/src/cf_file.c

Timestamp:

11/26/13 09:06:48 (10 years ago)

Author:

cheese

Message:

#1 fix memory leakage

File:

• trunk/src/cf_file.c (modified) (3 diffs)

trunk/src/cf_file.c

@@ -234 +234 @@
 CF_File_GetSize (const char * path)
 {
-    int length = 0;
     int result = 0;
 
@@ -241 +240 @@
     ASSERT_ARGS (path == NULL);
 
-    result = CF_File_Open ((cf_ctx *)&context, path, CF_FILE_READ);
-    if (result < 0)
-        return CF_ERROR_FILE_OPEN;
-
-    length = (int) lseek (context->fd, 0, SEEK_END);
-    if (length < 0)
-        return CF_ERROR_FILE_GET_SIZE;
+    TRY
+    {
+        result = CF_File_Open ((cf_ctx *)&context, path, CF_FILE_READ);
+        if (result < 0)
+        {
+            result = CF_ERROR_FILE_OPEN;
+            TRY_BREAK;
+        }
+
+        result = (int) lseek (context->fd, 0, SEEK_END);
+        if (result < 0)
+        {
+            result = CF_ERROR_FILE_GET_SIZE;
+            TRY_BREAK;
+        }
+    }
+    NO_CATCH;
 
     CF_File_Close ((cf_ctx)context);
 
-    return length;
+    return result;
 }
 
@@ -287 +296 @@
     ASSERT_ARGS (path == NULL);
 
-    length = strlen (path);
-    fullpath = (char *) calloc (length + 2, 1);
-    if (!fullpath)
-        return CF_ERROR_FILE_ALLOCATE_BUFFER;
-
-    f = fullpath;
-    d = stepPath;
-
-    snprintf (fullpath, length + 1, "%s%c", path, DELIMITER);
-
-    for (*d++ = *f++ ; *f ; *d++ = *f++)
+    TRY
     {
-        if (*f != DELIMITER)
-            continue;
-
-        if (CF_File_Exists (stepPath))
-            continue;
+        length = strlen (path);
+        fullpath = (char *) calloc (length + 2, 1);
+        if (!fullpath)
+        {
+            result = CF_ERROR_FILE_ALLOCATE_BUFFER;
+            TRY_BREAK;
+        }
+
+        f = fullpath;
+        d = stepPath;
+
+        snprintf (fullpath, length + 1, "%s%c", path, DELIMITER);
+
+        for (*d++ = *f++ ; *f ; *d++ = *f++)
+        {
+            if (*f != DELIMITER)
+                continue;
+
+            if (CF_File_Exists (stepPath))
+                continue;
 
 #define DIRECTORY_MODE  (S_IRWXU|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
-        result = mkdir (stepPath, DIRECTORY_MODE);
-        if (result && errno != EEXIST)
-            return CF_ERROR_FILE_MAKE_DIRECTORY;
+            result = mkdir (stepPath, DIRECTORY_MODE);
+            if (result && errno != EEXIST)
+            {
+                result = CF_ERROR_FILE_MAKE_DIRECTORY;
+                break;
+            }
+        }
     }
-
-    return CF_OK;
+    NO_CATCH;
+
+    if (fullpath)   free (fullpath);
+
+    return result;
 }