make app_queue 1.2 jump compliant (issue #5580)
[asterisk/asterisk.git] / file.c
diff --git a/file.c b/file.c
index d23879e..530ee48 100755 (executable)
--- a/file.c
+++ b/file.c
@@ -1,26 +1,28 @@
 /*
- * Asterisk -- A telephony toolkit for Linux.
+ * Asterisk -- An open source telephony toolkit.
  *
- * Generic File Format Support.
- * 
- * Copyright (C) 1999, Mark Spencer
+ * Copyright (C) 1999 - 2005, Digium, Inc.
+ *
+ * Mark Spencer <markster@digium.com>
  *
- * Mark Spencer <markster@linux-support.net>
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
  *
  * This program is free software, distributed under the terms of
- * the GNU General Public License
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
+ */
+
+/*! \file
+ *
+ * \brief Generic File Format Support.
+ * 
  */
 
 #include <sys/types.h>
-#include <asterisk/frame.h>
-#include <asterisk/file.h>
-#include <asterisk/logger.h>
-#include <asterisk/channel.h>
-#include <asterisk/sched.h>
-#include <asterisk/options.h>
-#include <asterisk/translate.h>
-#include <asterisk/utils.h>
-#include <asterisk/lock.h>
 #include <errno.h>
 #include <unistd.h>
 #include <stdlib.h>
 #include <dirent.h>
 #include <sys/types.h>
 #include <sys/stat.h>
+
 #include "asterisk.h"
-#include "astconf.h"
+
+ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+
+#include "asterisk/frame.h"
+#include "asterisk/file.h"
+#include "asterisk/cli.h"
+#include "asterisk/logger.h"
+#include "asterisk/channel.h"
+#include "asterisk/sched.h"
+#include "asterisk/options.h"
+#include "asterisk/translate.h"
+#include "asterisk/utils.h"
+#include "asterisk/lock.h"
+#include "asterisk/app.h"
+#include "asterisk/pbx.h"
 
 struct ast_format {
        /* Name of format */
@@ -42,9 +59,9 @@ struct ast_format {
        /* Format of frames it uses/provides (one only) */
        int format;
        /* Open an input stream, and start playback */
-       struct ast_filestream * (*open)(int fd);
+       struct ast_filestream * (*open)(FILE * f);
        /* Open an output stream, of a given file descriptor and comment it appropriately if applicable */
-       struct ast_filestream * (*rewrite)(int fd, char *comment);
+       struct ast_filestream * (*rewrite)(FILE *f, const char *comment);
        /* Write a frame to a channel */
        int (*write)(struct ast_filestream *, struct ast_frame *);
        /* seek num samples into file, whence(think normal seek) */
@@ -70,6 +87,7 @@ struct ast_filestream {
        int flags;
        mode_t mode;
        char *filename;
+       char *realfilename;
        /* Video file stream */
        struct ast_filestream *vfs;
        /* Transparently translate from another format -- just once */
@@ -84,9 +102,9 @@ AST_MUTEX_DEFINE_STATIC(formatlock);
 
 static struct ast_format *formats = NULL;
 
-int ast_format_register(char *name, char *exts, int format,
-                                               struct ast_filestream * (*open)(int fd),
-                                               struct ast_filestream * (*rewrite)(int fd, char *comment),
+int ast_format_register(const char *name, const char *exts, int format,
+                                               struct ast_filestream * (*open)(FILE *f),
+                                               struct ast_filestream * (*rewrite)(FILE *f, const char *comment),
                                                int (*write)(struct ast_filestream *, struct ast_frame *),
                                                int (*seek)(struct ast_filestream *, long sample_offset, int whence),
                                                int (*trunc)(struct ast_filestream *),
@@ -115,8 +133,8 @@ int ast_format_register(char *name, char *exts, int format,
                ast_mutex_unlock(&formatlock);
                return -1;
        }
-       strncpy(tmp->name, name, sizeof(tmp->name)-1);
-       strncpy(tmp->exts, exts, sizeof(tmp->exts)-1);
+       ast_copy_string(tmp->name, name, sizeof(tmp->name));
+       ast_copy_string(tmp->exts, exts, sizeof(tmp->exts));
        tmp->open = open;
        tmp->rewrite = rewrite;
        tmp->read = read;
@@ -135,7 +153,7 @@ int ast_format_register(char *name, char *exts, int format,
        return 0;
 }
 
-int ast_format_unregister(char *name)
+int ast_format_unregister(const char *name)
 {
        struct ast_format *tmp, *tmpl = NULL;
        if (ast_mutex_lock(&formatlock)) {
@@ -185,7 +203,7 @@ int ast_writestream(struct ast_filestream *fs, struct ast_frame *f)
                        /* This is the audio portion.  Call the video one... */
                        if (!fs->vfs && fs->filename) {
                                /* XXX Support other video formats XXX */
-                               char *type = "h263";
+                               const char *type = "h263";
                                fs->vfs = ast_writefile(fs->filename, type, NULL, fs->flags, 0, fs->mode);
                                ast_log(LOG_DEBUG, "Opened video output file\n");
                        }
@@ -235,7 +253,7 @@ int ast_writestream(struct ast_filestream *fs, struct ast_frame *f)
        }
 }
 
-static int copy(char *infile, char *outfile)
+static int copy(const char *infile, const char *outfile)
 {
        int ifd;
        int ofd;
@@ -275,28 +293,41 @@ static int copy(char *infile, char *outfile)
        return 0;
 }
 
-static char *build_filename(char *filename, char *ext)
+static char *build_filename(const char *filename, const char *ext)
 {
-       char *fn;
-       char tmp[AST_CONFIG_MAX_PATH]="";
-       snprintf(tmp,sizeof(tmp)-1,"%s/%s",(char *)ast_config_AST_VAR_DIR,"sounds");
-       fn = malloc(strlen(tmp) + strlen(filename) + strlen(ext) + 10);
-       if (fn) {
-               if (filename[0] == '/') 
-                       sprintf(fn, "%s.%s", filename, ext);
-               else
-                       sprintf(fn, "%s/%s.%s", (char *)tmp, filename, ext);
+       char *fn, type[16];
+       int fnsize = 0;
+
+       if (!strcmp(ext, "wav49")) {
+               ast_copy_string(type, "WAV", sizeof(type));
+       } else {
+               ast_copy_string(type, ext, sizeof(type));
+       }
+
+       if (filename[0] == '/') {
+               fnsize = strlen(filename) + strlen(type) + 2;
+               fn = malloc(fnsize);
+               if (fn)
+                       snprintf(fn, fnsize, "%s.%s", filename, type);
+       } else {
+               char tmp[AST_CONFIG_MAX_PATH] = "";
+
+               snprintf(tmp, sizeof(tmp), "%s/%s", ast_config_AST_VAR_DIR, "sounds");
+               fnsize = strlen(tmp) + strlen(filename) + strlen(type) + 3;
+               fn = malloc(fnsize);
+               if (fn)
+                       snprintf(fn, fnsize, "%s/%s.%s", tmp, filename, type);
        }
+
        return fn;
-       
 }
 
-static int exts_compare(char *exts, char *type)
+static int exts_compare(const char *exts, const char *type)
 {
        char *stringp = NULL, *ext;
        char tmp[256];
 
-       strncpy(tmp, exts, sizeof(tmp) - 1);
+       ast_copy_string(tmp, exts, sizeof(tmp));
        stringp = tmp;
        while ((ext = strsep(&stringp, "|"))) {
                if (!strcmp(ext, type)) {
@@ -313,13 +344,14 @@ static int exts_compare(char *exts, char *type)
 #define ACTION_OPEN   4
 #define ACTION_COPY   5
 
-static int ast_filehelper(char *filename, char *filename2, char *fmt, int action)
+static int ast_filehelper(const char *filename, const char *filename2, const char *fmt, int action)
 {
        struct stat st;
        struct ast_format *f;
        struct ast_filestream *s;
        int res=0, ret = 0;
        char *ext=NULL, *exts, *fn, *nfn;
+       FILE *bfile;
        struct ast_channel *chan = (struct ast_channel *)filename2;
        
        /* Start with negative response */
@@ -341,13 +373,10 @@ static int ast_filehelper(char *filename, char *filename2, char *fmt, int action
        while(f) {
                if (!fmt || exts_compare(f->exts, fmt)) {
                        char *stringp=NULL;
-                       exts = strdup(f->exts);
+                       exts = ast_strdupa(f->exts);
                        /* Try each kind of extension */
                        stringp=exts;
                        ext = strsep(&stringp, "|");
-                       if (!strcmp(ext,"wav49")) {
-                               ext = "WAV";
-                       }
                        do {
                                fn = build_filename(filename, ext);
                                if (fn) {
@@ -385,9 +414,10 @@ static int ast_filehelper(char *filename, char *filename2, char *fmt, int action
                                                case ACTION_OPEN:
                                                        if ((ret < 0) && ((chan->writeformat & f->format) ||
                                                                                ((f->format >= AST_FORMAT_MAX_AUDIO) && fmt))) {
-                                                               ret = open(fn, O_RDONLY);
-                                                               if (ret >= 0) {
-                                                                       s = f->open(ret);
+                                                               bfile = fopen(fn, "r");
+                                                               if (bfile) {
+                                                                       ret = 1;
+                                                                       s = f->open(bfile);
                                                                        if (s) {
                                                                                s->lasttimeout = -1;
                                                                                s->fmt = f;
@@ -398,11 +428,14 @@ static int ast_filehelper(char *filename, char *filename2, char *fmt, int action
                                                                                else
                                                                                        chan->vstream = s;
                                                                        } else {
-                                                                               close(ret);
-                                                                               ast_log(LOG_WARNING, "Unable to open fd on %s\n", fn);
+                                                                               fclose(bfile);
+                                                                               ast_log(LOG_WARNING, "Unable to open file on %s\n", fn);
+                                                                               ret = -1;
                                                                        }
-                                                               } else
+                                                               } else{
                                                                        ast_log(LOG_WARNING, "Couldn't open file %s\n", fn);
+                                                                       ret = -1;
+                                                               }
                                                        }
                                                        break;
                                                default:
@@ -416,7 +449,7 @@ static int ast_filehelper(char *filename, char *filename2, char *fmt, int action
                                }
                                ext = strsep(&stringp, "|");
                        } while(ext);
-                       free(exts);
+                       
                }
                f = f->next;
        }
@@ -425,8 +458,12 @@ static int ast_filehelper(char *filename, char *filename2, char *fmt, int action
                res = ret ? ret : -1;
        return res;
 }
+struct ast_filestream *ast_openstream(struct ast_channel *chan, const char *filename, const char *preflang)
+{
+       return ast_openstream_full(chan, filename, preflang, 0);
+}
 
-struct ast_filestream *ast_openstream(struct ast_channel *chan, char *filename, char *preflang)
+struct ast_filestream *ast_openstream_full(struct ast_channel *chan, const char *filename, const char *preflang, int asis)
 {
        /* This is a fairly complex routine.  Essentially we should do 
           the following:
@@ -440,18 +477,20 @@ struct ast_filestream *ast_openstream(struct ast_channel *chan, char *filename,
               set it up.
                   
        */
-       int fd = -1;
        int fmts = -1;
        char filename2[256]="";
-       char filename3[256]="";
+       char filename3[256];
        char *endpart;
        int res;
-       ast_stopstream(chan);
-       /* do this first, otherwise we detect the wrong writeformat */
-       if (chan->generator)
-               ast_deactivate_generator(chan);
-       if (preflang && !ast_strlen_zero(preflang)) {
-               strncpy(filename3, filename, sizeof(filename3) - 1);
+
+       if (!asis) {
+               /* do this first, otherwise we detect the wrong writeformat */
+               ast_stopstream(chan);
+               if (chan->generator)
+                       ast_deactivate_generator(chan);
+       }
+       if (!ast_strlen_zero(preflang)) {
+               ast_copy_string(filename3, filename, sizeof(filename3));
                endpart = strrchr(filename3, '/');
                if (endpart) {
                        *endpart = '\0';
@@ -462,7 +501,7 @@ struct ast_filestream *ast_openstream(struct ast_channel *chan, char *filename,
                fmts = ast_fileexists(filename2, NULL, NULL);
        }
        if (fmts < 1) {
-               strncpy(filename2, filename, sizeof(filename2)-1);
+               ast_copy_string(filename2, filename, sizeof(filename2));
                fmts = ast_fileexists(filename2, NULL, NULL);
        }
        if (fmts < 1) {
@@ -473,13 +512,13 @@ struct ast_filestream *ast_openstream(struct ast_channel *chan, char *filename,
        /* Set the channel to a format we can work with */
        res = ast_set_write_format(chan, fmts);
        
-       fd = ast_filehelper(filename2, (char *)chan, NULL, ACTION_OPEN);
-       if(fd >= 0)
+       res = ast_filehelper(filename2, (char *)chan, NULL, ACTION_OPEN);
+       if (res >= 0)
                return chan->stream;
        return NULL;
 }
 
-struct ast_filestream *ast_openvstream(struct ast_channel *chan, char *filename, char *preflang)
+struct ast_filestream *ast_openvstream(struct ast_channel *chan, const char *filename, const char *preflang)
 {
        /* This is a fairly complex routine.  Essentially we should do 
           the following:
@@ -499,24 +538,24 @@ struct ast_filestream *ast_openvstream(struct ast_channel *chan, char *filename,
        char lang2[MAX_LANGUAGE];
        /* XXX H.263 only XXX */
        char *fmt = "h263";
-       if (preflang && !ast_strlen_zero(preflang)) {
+       if (!ast_strlen_zero(preflang)) {
                snprintf(filename2, sizeof(filename2), "%s/%s", preflang, filename);
                fmts = ast_fileexists(filename2, fmt, NULL);
                if (fmts < 1) {
-                       strncpy(lang2, preflang, sizeof(lang2)-1);
+                       ast_copy_string(lang2, preflang, sizeof(lang2));
                        snprintf(filename2, sizeof(filename2), "%s/%s", lang2, filename);
                        fmts = ast_fileexists(filename2, fmt, NULL);
                }
        }
        if (fmts < 1) {
-               strncpy(filename2, filename, sizeof(filename2)-1);
+               ast_copy_string(filename2, filename, sizeof(filename2));
                fmts = ast_fileexists(filename2, fmt, NULL);
        }
        if (fmts < 1) {
                return NULL;
        }
        fd = ast_filehelper(filename2, (char *)chan, fmt, ACTION_OPEN);
-       if(fd >= 0)
+       if (fd >= 0)
                return chan->vstream;
        ast_log(LOG_WARNING, "File %s has video but couldn't be opened\n", filename);
        return NULL;
@@ -645,6 +684,8 @@ int ast_stream_rewind(struct ast_filestream *fs, long ms)
 
 int ast_closestream(struct ast_filestream *f)
 {
+       char *cmd = NULL;
+       size_t size = 0;
        /* Stop a running stream if there is one */
        if (f->owner) {
                if (f->fmt->format < AST_FORMAT_MAX_AUDIO) {
@@ -667,15 +708,29 @@ int ast_closestream(struct ast_filestream *f)
                ast_translator_free_path(f->trans);
                f->trans = NULL;
        }
-       if (f->filename)
+
+       if (f->realfilename && f->filename) {
+                       size = strlen(f->filename) + strlen(f->realfilename) + 15;
+                       cmd = alloca(size);
+                       memset(cmd,0,size);
+                       snprintf(cmd,size,"/bin/mv -f %s %s",f->filename,f->realfilename);
+                       ast_safe_system(cmd);
+       }
+
+       if (f->filename) {
                free(f->filename);
-       f->filename = NULL;
+               f->filename = NULL;
+       }
+       if (f->realfilename) {
+               free(f->realfilename);
+               f->realfilename = NULL;
+       }
        f->fmt->close(f);
        return 0;
 }
 
 
-int ast_fileexists(char *filename, char *fmt, char *preflang)
+int ast_fileexists(const char *filename, const char *fmt, const char *preflang)
 {
        char filename2[256];
        char tmp[256];
@@ -684,53 +739,61 @@ int ast_fileexists(char *filename, char *fmt, char *preflang)
        char *c;
        char lang2[MAX_LANGUAGE];
        int res = -1;
-       if (preflang && !ast_strlen_zero(preflang)) {
+       if (!ast_strlen_zero(preflang)) {
                /* Insert the language between the last two parts of the path */
-               strncpy(tmp, filename, sizeof(tmp) - 1);
+               ast_copy_string(tmp, filename, sizeof(tmp));
                c = strrchr(tmp, '/');
                if (c) {
                        *c = '\0';
                        postfix = c+1;
                        prefix = tmp;
+                       snprintf(filename2, sizeof(filename2), "%s/%s/%s", prefix, preflang, postfix);
                } else {
                        postfix = tmp;
                        prefix="";
+                       snprintf(filename2, sizeof(filename2), "%s/%s", preflang, postfix);
                }
-               snprintf(filename2, sizeof(filename2), "%s/%s/%s", prefix, preflang, postfix);
                res = ast_filehelper(filename2, NULL, fmt, ACTION_EXISTS);
                if (res < 1) {
                        char *stringp=NULL;
-                       strncpy(lang2, preflang, sizeof(lang2)-1);
+                       ast_copy_string(lang2, preflang, sizeof(lang2));
                        stringp=lang2;
                        strsep(&stringp, "_");
+                       /* If language is a specific locality of a language (like es_MX), strip the locality and try again */
                        if (strcmp(lang2, preflang)) {
-                               snprintf(filename2, sizeof(filename2), "%s/%s/%s", prefix, lang2, postfix);
+                               if (ast_strlen_zero(prefix)) {
+                                       snprintf(filename2, sizeof(filename2), "%s/%s", lang2, postfix);
+                               } else {
+                                       snprintf(filename2, sizeof(filename2), "%s/%s/%s", prefix, lang2, postfix);
+                               }
                                res = ast_filehelper(filename2, NULL, fmt, ACTION_EXISTS);
                        }
                }
        }
+
+       /* Fallback to no language (usually winds up being American English) */
        if (res < 1) {
                res = ast_filehelper(filename, NULL, fmt, ACTION_EXISTS);
        }
        return res;
 }
 
-int ast_filedelete(char *filename, char *fmt)
+int ast_filedelete(const char *filename, const char *fmt)
 {
        return ast_filehelper(filename, NULL, fmt, ACTION_DELETE);
 }
 
-int ast_filerename(char *filename, char *filename2, char *fmt)
+int ast_filerename(const char *filename, const char *filename2, const char *fmt)
 {
        return ast_filehelper(filename, filename2, fmt, ACTION_RENAME);
 }
 
-int ast_filecopy(char *filename, char *filename2, char *fmt)
+int ast_filecopy(const char *filename, const char *filename2, const char *fmt)
 {
        return ast_filehelper(filename, filename2, fmt, ACTION_COPY);
 }
 
-int ast_streamfile(struct ast_channel *chan, char *filename, char *preflang)
+int ast_streamfile(struct ast_channel *chan, const char *filename, const char *preflang)
 {
        struct ast_filestream *fs;
        struct ast_filestream *vfs;
@@ -739,14 +802,14 @@ int ast_streamfile(struct ast_channel *chan, char *filename, char *preflang)
        vfs = ast_openvstream(chan, filename, preflang);
        if (vfs)
                ast_log(LOG_DEBUG, "Ooh, found a video stream, too\n");
-       if(fs){
-               if(ast_applystream(chan, fs))
+       if (fs){
+               if (ast_applystream(chan, fs))
                        return -1;
-               if(vfs && ast_applystream(chan, vfs))
+               if (vfs && ast_applystream(chan, vfs))
                        return -1;
-               if(ast_playstream(fs))
+               if (ast_playstream(fs))
                        return -1;
-               if(vfs && ast_playstream(vfs))
+               if (vfs && ast_playstream(vfs))
                        return -1;
 #if 1
                if (option_verbose > 2)
@@ -758,115 +821,171 @@ int ast_streamfile(struct ast_channel *chan, char *filename, char *preflang)
        return -1;
 }
 
-struct ast_filestream *ast_readfile(char *filename, char *type, char *comment, int flags, int check, mode_t mode)
+struct ast_filestream *ast_readfile(const char *filename, const char *type, const char *comment, int flags, int check, mode_t mode)
 {
-       int fd,myflags = 0;
+       FILE *bfile;
        struct ast_format *f;
-       struct ast_filestream *fs=NULL;
+       struct ast_filestream *fs = NULL;
        char *fn;
-       char *ext;
+
        if (ast_mutex_lock(&formatlock)) {
                ast_log(LOG_WARNING, "Unable to lock format list\n");
                return NULL;
        }
-       f = formats;
-       while(f) {
-               if (exts_compare(f->exts, type)) {
-                       char *stringp=NULL;
-                       /* XXX Implement check XXX */
-                       ext = strdup(f->exts);
-                       stringp=ext;
-                       ext = strsep(&stringp, "|");
-                       fn = build_filename(filename, ext);
-                       fd = open(fn, flags | myflags);
-                       if (fd >= 0) {
-                               errno = 0;
-                               if ((fs = f->open(fd))) {
-                                       fs->trans = NULL;
-                                       fs->fmt = f;
-                                       fs->flags = flags;
-                                       fs->mode = mode;
-                                       fs->filename = strdup(filename);
-                                       fs->vfs = NULL;
-                               } else {
-                                       ast_log(LOG_WARNING, "Unable to open %s\n", fn);
-                                       close(fd);
-                                       unlink(fn);
-                               }
-                       } else if (errno != EEXIST)
-                               ast_log(LOG_WARNING, "Unable to open file %s: %s\n", fn, strerror(errno));
-                       free(fn);
-                       free(ext);
-                       break;
-               }
-               f = f->next;
+
+       for (f = formats; f && !fs; f = f->next) {
+               if (!exts_compare(f->exts, type))
+                       continue;
+
+               fn = build_filename(filename, type);
+               bfile = fopen(fn, "r");
+               if (bfile) {
+                       errno = 0;
+
+                       if (!(fs = f->open(bfile))) {
+                               ast_log(LOG_WARNING, "Unable to open %s\n", fn);
+                               fclose(bfile);
+                               free(fn);
+                               continue;
+                       }
+
+                       fs->trans = NULL;
+                       fs->fmt = f;
+                       fs->flags = flags;
+                       fs->mode = mode;
+                       fs->filename = strdup(filename);
+                       fs->vfs = NULL;
+               } else if (errno != EEXIST)
+                       ast_log(LOG_WARNING, "Unable to open file %s: %s\n", fn, strerror(errno));
+               free(fn);
        }
+
        ast_mutex_unlock(&formatlock);
-       if (!f) 
+       if (!fs) 
                ast_log(LOG_WARNING, "No such format '%s'\n", type);
+
        return fs;
 }
 
-struct ast_filestream *ast_writefile(char *filename, char *type, char *comment, int flags, int check, mode_t mode)
+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;
+       int fd, myflags = 0;
+       /* compiler claims this variable can be used before initialization... */
+       FILE *bfile = NULL;
        struct ast_format *f;
-       struct ast_filestream *fs=NULL;
-       char *fn;
-       char *ext;
+       struct ast_filestream *fs = NULL;
+       char *fn, *orig_fn = NULL;
+       char *buf = NULL;
+       size_t size = 0;
+
        if (ast_mutex_lock(&formatlock)) {
                ast_log(LOG_WARNING, "Unable to lock format list\n");
                return NULL;
        }
+
        /* set the O_TRUNC flag if and only if there is no O_APPEND specified */
-       if (!(flags & O_APPEND)) 
+       if (flags & O_APPEND) { 
+               /* We really can't use O_APPEND as it will break WAV header updates */
+               flags &= ~O_APPEND;
+       } else {
                myflags = O_TRUNC;
+       }
        
        myflags |= O_WRONLY | O_CREAT;
 
-       f = formats;
-       while(f) {
-               if (exts_compare(f->exts, type)) {
-                       char *stringp=NULL;
-                       /* XXX Implement check XXX */
-                       ext = strdup(f->exts);
-                       stringp=ext;
-                       ext = strsep(&stringp, "|");
-                       fn = build_filename(filename, ext);
+       for (f = formats; f && !fs; f = f->next) {
+               if (!exts_compare(f->exts, type))
+                       continue;
+
+               fn = build_filename(filename, type);
+               fd = open(fn, flags | myflags, mode);
+               if (fd > -1) {
+                       /* fdopen() the resulting file stream */
+                       bfile = fdopen(fd, ((flags | myflags) & O_RDWR) ? "w+" : "w");
+                       if (!bfile) {
+                               ast_log(LOG_WARNING, "Whoa, fdopen failed: %s!\n", strerror(errno));
+                               close(fd);
+                               fd = -1;
+                       }
+               }
+               
+               if (option_cache_record_files && (fd > -1)) {
+                       char *c;
+
+                       fclose(bfile);
+                       /*
+                         We touch orig_fn just as a place-holder so other things (like vmail) see the file is there.
+                         What we are really doing is writing to record_cache_dir until we are done then we will mv the file into place.
+                       */
+                       orig_fn = ast_strdupa(fn);
+                       for (c = fn; *c; c++)
+                               if (*c == '/')
+                                       *c = '_';
+
+                       size = strlen(fn) + strlen(record_cache_dir) + 2;
+                       buf = alloca(size);
+                       strcpy(buf, record_cache_dir);
+                       strcat(buf, "/");
+                       strcat(buf, fn);
+                       free(fn);
+                       fn = buf;
                        fd = open(fn, flags | myflags, mode);
-                       if (fd >= 0) {
-                               errno = 0;
-                               if ((fs = f->rewrite(fd, comment))) {
-                                       fs->trans = NULL;
-                                       fs->fmt = f;
-                                       fs->flags = flags;
-                                       fs->mode = mode;
-                                       fs->filename = strdup(filename);
-                                       fs->vfs = NULL;
-                               } else {
-                                       ast_log(LOG_WARNING, "Unable to rewrite %s\n", fn);
+                       if (fd > -1) {
+                               /* fdopen() the resulting file stream */
+                               bfile = fdopen(fd, ((flags | myflags) & O_RDWR) ? "w+" : "w");
+                               if (!bfile) {
+                                       ast_log(LOG_WARNING, "Whoa, fdopen failed: %s!\n", strerror(errno));
                                        close(fd);
+                                       fd = -1;
+                               }
+                       }
+               }
+               if (fd > -1) {
+                       errno = 0;
+                       if ((fs = f->rewrite(bfile, comment))) {
+                               fs->trans = NULL;
+                               fs->fmt = f;
+                               fs->flags = flags;
+                               fs->mode = mode;
+                               if (orig_fn) {
+                                       fs->realfilename = strdup(orig_fn);
+                                       fs->filename = strdup(fn);
+                               } else {
+                                       fs->realfilename = NULL;
+                                       fs->filename = strdup(filename);
+                               }
+                               fs->vfs = NULL;
+                       } else {
+                               ast_log(LOG_WARNING, "Unable to rewrite %s\n", fn);
+                               close(fd);
+                               if (orig_fn) {
                                        unlink(fn);
+                                       unlink(orig_fn);
                                }
-                       } else if (errno != EEXIST)
-                               ast_log(LOG_WARNING, "Unable to open file %s: %s\n", fn, strerror(errno));
-                       free(fn);
-                       free(ext);
-                       break;
+                       }
+               } else if (errno != EEXIST) {
+                       ast_log(LOG_WARNING, "Unable to open file %s: %s\n", fn, strerror(errno));
+                       if (orig_fn)
+                               unlink(orig_fn);
                }
-               f = f->next;
+               /* if buf != NULL then fn is already free and pointing to it */
+               if (!buf)
+                       free(fn);
        }
+
        ast_mutex_unlock(&formatlock);
-       if (!f) 
+       if (!fs)
                ast_log(LOG_WARNING, "No such format '%s'\n", type);
+
        return fs;
 }
 
-char ast_waitstream(struct ast_channel *c, char *breakon)
+int ast_waitstream(struct ast_channel *c, const char *breakon)
 {
        /* XXX Maybe I should just front-end ast_waitstream_full ? XXX */
        int res;
        struct ast_frame *fr;
+       if (!breakon) breakon = "";
        while(c->stream) {
                res = ast_sched_wait(c->sched);
                if ((res < 0) && !c->timingfunc) {
@@ -903,6 +1022,7 @@ char ast_waitstream(struct ast_channel *c, char *breakon)
                                        return -1;
                                case AST_CONTROL_RINGING:
                                case AST_CONTROL_ANSWER:
+                               case AST_CONTROL_VIDUPDATE:
                                        /* Unimportant */
                                        break;
                                default:
@@ -917,10 +1037,18 @@ char ast_waitstream(struct ast_channel *c, char *breakon)
        return (c->_softhangup ? -1 : 0);
 }
 
-char ast_waitstream_fr(struct ast_channel *c, char *breakon, char *forward, char *rewind, int ms)
+int ast_waitstream_fr(struct ast_channel *c, const char *breakon, const char *forward, const char *rewind, int ms)
 {
        int res;
        struct ast_frame *fr;
+
+       if (!breakon)
+                       breakon = "";
+       if (!forward)
+                       forward = "";
+       if (!rewind)
+                       rewind = "";
+       
        while(c->stream) {
                res = ast_sched_wait(c->sched);
                if ((res < 0) && !c->timingfunc) {
@@ -978,13 +1106,16 @@ char ast_waitstream_fr(struct ast_channel *c, char *breakon, char *forward, char
        return (c->_softhangup ? -1 : 0);
 }
 
-char ast_waitstream_full(struct ast_channel *c, char *breakon, int audiofd, int cmdfd)
+int ast_waitstream_full(struct ast_channel *c, const char *breakon, int audiofd, int cmdfd)
 {
        int res;
        int ms;
        int outfd;
        struct ast_frame *fr;
        struct ast_channel *rchan;
+
+       if (!breakon)
+               breakon = "";
        
        while(c->stream) {
                ms = ast_sched_wait(c->sched);
@@ -996,6 +1127,9 @@ char ast_waitstream_full(struct ast_channel *c, char *breakon, int audiofd, int
                        ms = 1000;
                rchan = ast_waitfor_nandfds(&c, 1, &cmdfd, (cmdfd > -1) ? 1 : 0, NULL, &outfd, &ms);
                if (!rchan && (outfd < 0) && (ms)) {
+                       /* Continue */
+                       if (errno == EINTR)
+                               continue;
                        ast_log(LOG_WARNING, "Wait failed (%s)\n", strerror(errno));
                        return -1;
                } else if (outfd > -1) {
@@ -1039,8 +1173,112 @@ char ast_waitstream_full(struct ast_channel *c, char *breakon, int audiofd, int
                        ast_frfree(fr);
                }
                ast_sched_runq(c->sched);
-       
-               
        }
        return (c->_softhangup ? -1 : 0);
 }
+
+int ast_waitstream_exten(struct ast_channel *c, const char *context)
+{
+       /* Waitstream, with return in the case of a valid 1 digit extension */
+       /* in the current or specified context being pressed */
+       /* XXX Maybe I should just front-end ast_waitstream_full ? XXX */
+       int res;
+       struct ast_frame *fr;
+       char exten[AST_MAX_EXTENSION];
+
+       if (!context) context = c->context;
+       while(c->stream) {
+               res = ast_sched_wait(c->sched);
+               if ((res < 0) && !c->timingfunc) {
+                       ast_stopstream(c);
+                       break;
+               }
+               if (res < 0)
+                       res = 1000;
+               res = ast_waitfor(c, res);
+               if (res < 0) {
+                       ast_log(LOG_WARNING, "Select failed (%s)\n", strerror(errno));
+                       return res;
+               } else if (res > 0) {
+                       fr = ast_read(c);
+                       if (!fr) {
+#if 0
+                               ast_log(LOG_DEBUG, "Got hung up\n");
+#endif
+                               return -1;
+                       }
+                       
+                       switch(fr->frametype) {
+                       case AST_FRAME_DTMF:
+                               res = fr->subclass;
+                               snprintf(exten, sizeof(exten), "%c", res);
+                               if (ast_exists_extension(c, context, exten, 1, c->cid.cid_num)) {
+                                       ast_frfree(fr);
+                                       return res;
+                               }
+                               break;
+                       case AST_FRAME_CONTROL:
+                               switch(fr->subclass) {
+                               case AST_CONTROL_HANGUP:
+                                       ast_frfree(fr);
+                                       return -1;
+                               case AST_CONTROL_RINGING:
+                               case AST_CONTROL_ANSWER:
+                                       /* Unimportant */
+                                       break;
+                               default:
+                                       ast_log(LOG_WARNING, "Unexpected control subclass '%d'\n", fr->subclass);
+                               }
+                       }
+                       /* Ignore */
+                       ast_frfree(fr);
+               }
+               ast_sched_runq(c->sched);
+       }
+       return (c->_softhangup ? -1 : 0);
+}
+
+static int show_file_formats(int fd, int argc, char *argv[])
+{
+#define FORMAT "%-10s %-10s %-20s\n"
+#define FORMAT2 "%-10s %-10s %-20s\n"
+       struct ast_format *f;
+       int count_fmt = 0;
+
+       if (argc != 3)
+               return RESULT_SHOWUSAGE;
+       ast_cli(fd, FORMAT, "Format", "Name", "Extensions");
+               
+       if (ast_mutex_lock(&formatlock)) {
+               ast_log(LOG_WARNING, "Unable to lock format list\n");
+               return -1;
+       }
+
+       f = formats;
+       while(f) {
+               ast_cli(fd, FORMAT2, ast_getformatname(f->format), f->name, f->exts);
+               f = f->next;
+               count_fmt++;
+       };
+       ast_mutex_unlock(&formatlock);
+       ast_cli(fd, "%d file formats registered.\n", count_fmt);
+       return RESULT_SUCCESS;
+#undef FORMAT
+#undef FORMAT2
+       
+}
+
+struct ast_cli_entry show_file =
+{
+       { "show", "file", "formats" },
+       show_file_formats,
+       "Displays file formats",
+       "Usage: show file formats\n"
+       "       displays currently registered file formats (if any)\n"
+};
+
+int ast_file_init(void)
+{
+       ast_cli_register(&show_file);
+       return 0;
+}