minor changes and eliminate some compiler warnings
authorKevin P. Fleming <kpfleming@digium.com>
Sun, 16 Oct 2005 23:26:35 +0000 (23:26 +0000)
committerKevin P. Fleming <kpfleming@digium.com>
Sun, 16 Oct 2005 23:26:35 +0000 (23:26 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@6803 65c4cc65-6c06-0410-ace0-fbb531ad65f3

apps/app_mixmonitor.c
apps/app_muxmon.c
file.c

index 852eae8..73e5c5a 100755 (executable)
@@ -28,8 +28,6 @@
 #define get_volfactor(x) x ? ((x > 0) ? (1 << x) : ((1 << abs(x)) * -1)) : 0
 #define minmax(x,y) x ? (x > y) ? y : ((x < (y * -1)) ? (y * -1) : x) : 0 
 
-AST_MUTEX_DEFINE_STATIC(modlock);
-
 static char *tdesc = "Native Channel Monitoring Module";
 static char *app = "MuxMon";
 static char *synopsis = "Record A Call Natively";
@@ -460,7 +458,6 @@ static int muxmon_cli(int fd, int argc, char **argv)
 {
        char *op, *chan_name = NULL, *args = NULL;
        struct ast_channel *chan;
-       int count = 0;
 
        if (argc > 2) {
                op = argv[1];
index 852eae8..73e5c5a 100755 (executable)
@@ -28,8 +28,6 @@
 #define get_volfactor(x) x ? ((x > 0) ? (1 << x) : ((1 << abs(x)) * -1)) : 0
 #define minmax(x,y) x ? (x > y) ? y : ((x < (y * -1)) ? (y * -1) : x) : 0 
 
-AST_MUTEX_DEFINE_STATIC(modlock);
-
 static char *tdesc = "Native Channel Monitoring Module";
 static char *app = "MuxMon";
 static char *synopsis = "Record A Call Natively";
@@ -460,7 +458,6 @@ static int muxmon_cli(int fd, int argc, char **argv)
 {
        char *op, *chan_name = NULL, *args = NULL;
        struct ast_channel *chan;
-       int count = 0;
 
        if (argc > 2) {
                op = argv[1];
diff --git a/file.c b/file.c
index c4fac49..d48aaeb 100755 (executable)
--- a/file.c
+++ b/file.c
@@ -870,7 +870,8 @@ struct ast_filestream *ast_readfile(const char *filename, const char *type, cons
 struct ast_filestream *ast_writefile(const char *filename, const char *type, const char *comment, int flags, int check, mode_t mode)
 {
        int fd, myflags = 0;
-       FILE *bfile;
+       /* compiler claims this variable can be used before initialization... */
+       FILE *bfile = NULL;
        struct ast_format *f;
        struct ast_filestream *fs = NULL;
        char *fn, *orig_fn = NULL;
@@ -908,7 +909,7 @@ struct ast_filestream *ast_writefile(const char *filename, const char *type, con
                        }
                }
                
-               if (option_cache_record_files && fd >= 0) {
+               if (option_cache_record_files && (fd > -1)) {
                        char *c;
 
                        fclose(bfile);
@@ -923,8 +924,9 @@ struct ast_filestream *ast_writefile(const char *filename, const char *type, con
 
                        size = strlen(fn) + strlen(record_cache_dir) + 2;
                        buf = alloca(size);
-                       memset(buf, 0, size);
-                       snprintf(buf, size, "%s/%s", record_cache_dir, fn);
+                       strcpy(buf, record_cache_dir);
+                       strcat(buf, "/");
+                       strcat(buf, fn);
                        free(fn);
                        fn = buf;
                        fd = open(fn, flags | myflags, mode);
@@ -938,9 +940,8 @@ struct ast_filestream *ast_writefile(const char *filename, const char *type, con
                                }
                        }
                }
-               if (fd >= 0) {
+               if (fd > -1) {
                        errno = 0;
-
                        if ((fs = f->rewrite(bfile, comment))) {
                                fs->trans = NULL;
                                fs->fmt = f;