Merged revisions 167566 via svnmerge from
authorRussell Bryant <russell@russellbryant.com>
Wed, 7 Jan 2009 22:36:34 +0000 (22:36 +0000)
committerRussell Bryant <russell@russellbryant.com>
Wed, 7 Jan 2009 22:36:34 +0000 (22:36 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r167566 | russell | 2009-01-07 16:35:36 -0600 (Wed, 07 Jan 2009) | 2 lines

Fix the last couple of places where free() was improperly used directly.

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@167569 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/file.c

index ee9d5d9..995ba4a 100644 (file)
@@ -965,11 +965,11 @@ struct ast_filestream *ast_readfile(const char *filename, const char *type, cons
 
                if (!bfile || (fs = get_filestream(f, bfile)) == NULL || open_wrapper(fs) ) {
                        ast_log(LOG_WARNING, "Unable to open %s\n", fn);
-                       if (fs)
-                               ast_free(fs);
+                       if (fs) {
+                               ast_closestream(fs);
+                       }
                        fs = NULL;
-                       if (bfile)
-                               fclose(bfile);
+                       bfile = NULL;
                        ast_free(fn);
                        break;                          
                }
@@ -1079,9 +1079,10 @@ struct ast_filestream *ast_writefile(const char *filename, const char *type, con
                                        unlink(fn);
                                        unlink(orig_fn);
                                }
-                               if (fs)
-                                       ast_free(fs);
-                               fs = NULL;
+                               if (fs) {
+                                       ast_closestream(fs);
+                                       fs = NULL;
+                               }
                                continue;
                        }
                        fs->trans = NULL;