media formats: re-architect handling of media for performance improvements
[asterisk/asterisk.git] / formats / format_h263.c
index 16b090c..6511cef 100644 (file)
  * \brief Save to raw, headerless h263 data.
  * \arg File name extension: h263
  * \ingroup formats
+ * \arg See \ref AstVideo
  */
-#include <unistd.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
-#include <stdlib.h>
-#include <sys/time.h>
-#include <stdio.h>
-#include <errno.h>
-#include <string.h>
 
+/*** MODULEINFO
+       <support_level>core</support_level>
+ ***/
 #include "asterisk.h"
 
 ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 
-#include "asterisk/lock.h"
-#include "asterisk/channel.h"
-#include "asterisk/file.h"
-#include "asterisk/logger.h"
-#include "asterisk/sched.h"
+#include "asterisk/mod_format.h"
 #include "asterisk/module.h"
 #include "asterisk/endian.h"
+#include "asterisk/format_cache.h"
 
 /* Some Ideas for this code came from makeh263e.c by Jeffrey Chilton */
 
 /* Portions of the conversion code are by guido@sienanet.it */
 
-#define        BUF_SIZE        4096    /* Two Real h263 Frames */
+/* According to:
+ * http://lists.mpegif.org/pipermail/mp4-tech/2005-July/005741.html
+ * the maximum actual frame size is not 2048, but 8192.  Since the maximum
+ * theoretical limit is not much larger (32k = 15bits), we'll go for that
+ * size to ensure we don't corrupt frames sent to us (unless they're
+ * ridiculously large). */
+#define        BUF_SIZE        32768   /* Four real h.263 Frames */
+
+#define FRAME_ENDED 0x8000
 
 struct h263_desc {
        unsigned int lastts;
@@ -58,9 +59,8 @@ struct h263_desc {
 static int h263_open(struct ast_filestream *s)
 {
        unsigned int ts;
-       int res;
 
-       if ((res = fread(&ts, 1, sizeof(ts), s->f)) < sizeof(ts)) {
+       if (fread(&ts, 1, sizeof(ts), s->f) < sizeof(ts)) {
                ast_log(LOG_WARNING, "Empty file!\n");
                return -1;
        }
@@ -70,35 +70,30 @@ static int h263_open(struct ast_filestream *s)
 static struct ast_frame *h263_read(struct ast_filestream *s, int *whennext)
 {
        int res;
-       int mark;
+       uint32_t mark;
        unsigned short len;
        unsigned int ts;
-       struct h263_desc *fs = (struct h263_desc *)s->private;
+       struct h263_desc *fs = (struct h263_desc *)s->_private;
 
        /* Send a frame from the file to the appropriate channel */
        if ((res = fread(&len, 1, sizeof(len), s->f)) < 1)
                return NULL;
        len = ntohs(len);
-       mark = (len & 0x8000) ? 1 : 0;
+       mark = (len & FRAME_ENDED) ? 1 : 0;
        len &= 0x7fff;
        if (len > BUF_SIZE) {
                ast_log(LOG_WARNING, "Length %d is too long\n", len);
-               len = BUF_SIZE; /* XXX truncate ? */
+               return NULL;
        }
-       s->fr.frametype = AST_FRAME_VIDEO;
-       s->fr.subclass = AST_FORMAT_H263;
-       s->fr.mallocd = 0;
        AST_FRAME_SET_BUFFER(&s->fr, s->buf, AST_FRIENDLY_OFFSET, len);
-       if ((res = fread(s->fr.data, 1, s->fr.datalen, s->f)) != s->fr.datalen) {
+       if ((res = fread(s->fr.data.ptr, 1, s->fr.datalen, s->f)) != s->fr.datalen) {
                if (res)
                        ast_log(LOG_WARNING, "Short read (%d) (%s)!\n", res, strerror(errno));
                return NULL;
        }
        s->fr.samples = fs->lastts;     /* XXX what ? */
        s->fr.datalen = len;
-       s->fr.subclass |= mark;
-       s->fr.delivery.tv_sec = 0;
-       s->fr.delivery.tv_usec = 0;
+       s->fr.subclass.frame_ending = mark;
        if ((res = fread(&ts, 1, sizeof(ts), s->f)) == sizeof(ts)) {
                fs->lastts = ntohl(ts);
                *whennext = fs->lastts * 4/45;
@@ -112,20 +107,8 @@ static int h263_write(struct ast_filestream *fs, struct ast_frame *f)
        int res;
        unsigned int ts;
        unsigned short len;
-       int subclass;
-       int mark=0;
-       if (f->frametype != AST_FRAME_VIDEO) {
-               ast_log(LOG_WARNING, "Asked to write non-video frame!\n");
-               return -1;
-       }
-       subclass = f->subclass;
-       if (subclass & 0x1)
-               mark=0x8000;
-       subclass &= ~0x1;
-       if (subclass != AST_FORMAT_H263) {
-               ast_log(LOG_WARNING, "Asked to write non-h263 frame (%d)!\n", f->subclass);
-               return -1;
-       }
+       uint32_t mark = 0;
+       mark = f->subclass.frame_ending ? FRAME_ENDED : 0;
        ts = htonl(f->samples);
        if ((res = fwrite(&ts, 1, sizeof(ts), fs->f)) != sizeof(ts)) {
                        ast_log(LOG_WARNING, "Bad write (%d/4): %s\n", res, strerror(errno));
@@ -136,7 +119,7 @@ static int h263_write(struct ast_filestream *fs, struct ast_frame *f)
                        ast_log(LOG_WARNING, "Bad write (%d/2): %s\n", res, strerror(errno));
                        return -1;
        }
-       if ((res = fwrite(f->data, 1, f->datalen, fs->f)) != f->datalen) {
+       if ((res = fwrite(f->data.ptr, 1, f->datalen, fs->f)) != f->datalen) {
                        ast_log(LOG_WARNING, "Bad write (%d/%d): %s\n", res, f->datalen, strerror(errno));
                        return -1;
        }
@@ -151,10 +134,19 @@ static int h263_seek(struct ast_filestream *fs, off_t sample_offset, int whence)
 
 static int h263_trunc(struct ast_filestream *fs)
 {
-       /* Truncate file to current length */
-       if (ftruncate(fileno(fs->f), ftello(fs->f)) < 0)
+       int fd;
+       off_t cur;
+
+       if ((fd = fileno(fs->f)) < 0) {
+               ast_log(AST_LOG_WARNING, "Unable to determine file descriptor for h263 filestream %p: %s\n", fs, strerror(errno));
                return -1;
-       return 0;
+       }
+       if ((cur = ftello(fs->f)) < 0) {
+               ast_log(AST_LOG_WARNING, "Unable to determine current position in h263 filestream %p: %s\n", fs, strerror(errno));
+               return -1;
+       }
+       /* Truncate file to current length */
+       return ftruncate(fd, cur);
 }
 
 static off_t h263_tell(struct ast_filestream *fs)
@@ -163,10 +155,9 @@ static off_t h263_tell(struct ast_filestream *fs)
        return offset;  /* XXX totally bogus, needs fixing */
 }
 
-static const struct ast_format h263_f = {
+static struct ast_format_def h263_f = {
        .name = "h263",
        .exts = "h263",
-       .format = AST_FORMAT_H263,
        .open = h263_open,
        .write = h263_write,
        .seek = h263_seek,
@@ -175,27 +166,23 @@ static const struct ast_format h263_f = {
        .read = h263_read,
        .buf_size = BUF_SIZE + AST_FRIENDLY_OFFSET,
        .desc_size = sizeof(struct h263_desc),
-       .module = &mod_data,    /* XXX */
 };
 
-static int load_module(void *mod)
-{
-       return ast_format_register(&h263_f);
-}
-
-static int unload_module(void *mod)
-{
-       return ast_format_unregister(h263_f.name);
-}      
-
-static const char *description(void)
+static int load_module(void)
 {
-       return "Raw h263 data";
+       h263_f.format = ast_format_h263;
+       if (ast_format_def_register(&h263_f))
+               return AST_MODULE_LOAD_FAILURE;
+       return AST_MODULE_LOAD_SUCCESS;
 }
 
-static const char *key(void)
+static int unload_module(void)
 {
-       return ASTERISK_GPL_KEY;
+       return ast_format_def_unregister(h263_f.name);
 }
 
-STD_MOD1;
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "Raw H.263 data",
+       .load = load_module,
+       .unload = unload_module,
+       .load_pri = AST_MODPRI_APP_DEPEND
+);