Avoid cppcheck warnings; removing unused vars and a bit of cleanup.
[asterisk/asterisk.git] / formats / format_h263.c
index ee455be..ec08867 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Asterisk -- An open source telephony toolkit.
  *
- * Copyright (C) 1999 - 2005, Digium, Inc.
+ * Copyright (C) 1999 - 2006, Digium, Inc.
  *
  * Mark Spencer <markster@digium.com>
  *
  * \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"
 
@@ -48,7 +40,13 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 
 /* 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 */
 
 struct h263_desc {
        unsigned int lastts;
@@ -58,9 +56,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,10 +67,10 @@ 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)
@@ -83,20 +80,22 @@ static struct ast_frame *h263_read(struct ast_filestream *s, int *whennext)
        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;
+       ast_format_set(&s->fr.subclass.format, AST_FORMAT_H263, 0);
        s->fr.mallocd = 0;
-       FR_SET_BUF(&s->fr, s->buf, AST_FRIENDLY_OFFSET, len);
-       if ((res = fread(s->fr.data, 1, s->fr.datalen, s->f)) != s->fr.datalen) {
+       AST_FRAME_SET_BUFFER(&s->fr, s->buf, AST_FRIENDLY_OFFSET, len);
+       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;
+       if (mark) {
+               ast_format_set_video_mark(&s->fr.subclass.format);
+       }
        s->fr.delivery.tv_sec = 0;
        s->fr.delivery.tv_usec = 0;
        if ((res = fread(&ts, 1, sizeof(ts), s->f)) == sizeof(ts)) {
@@ -112,18 +111,14 @@ 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;
+       uint32_t 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);
+       mark = ast_format_get_video_mark(&f->subclass.format) ? 0x8000 : 0;
+       if (f->subclass.format.id != AST_FORMAT_H263) {
+               ast_log(LOG_WARNING, "Asked to write non-h263 frame (%s)!\n", ast_getformatname(&f->subclass.format));
                return -1;
        }
        ts = htonl(f->samples);
@@ -136,7 +131,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 +146,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,12 +167,9 @@ static off_t h263_tell(struct ast_filestream *fs)
        return offset;  /* XXX totally bogus, needs fixing */
 }
 
-static struct ast_format_lock me = { .usecnt = -1 };
-
-static const struct ast_format h263_f = {
+static struct ast_format_def h263_f = {
        .name = "h263",
-       .exts = "h264",
-       .format = AST_FORMAT_H263,
+       .exts = "h263",
        .open = h263_open,
        .write = h263_write,
        .seek = h263_seek,
@@ -177,30 +178,23 @@ static const struct ast_format h263_f = {
        .read = h263_read,
        .buf_size = BUF_SIZE + AST_FRIENDLY_OFFSET,
        .desc_size = sizeof(struct h263_desc),
-       .lockp = &me,
 };
 
-int load_module()
-{
-       return ast_format_register(&h263_f);
-}
-
-int unload_module()
-{
-       return ast_format_unregister(h263_f.name);
-}      
-
-int usecount()
+static int load_module(void)
 {
-       return me.usecnt;
+       ast_format_set(&h263_f.format, AST_FORMAT_H263, 0);
+       if (ast_format_def_register(&h263_f))
+               return AST_MODULE_LOAD_FAILURE;
+       return AST_MODULE_LOAD_SUCCESS;
 }
 
-const char *description()
+static int unload_module(void)
 {
-       return "Raw h263 data";
+       return ast_format_def_unregister(h263_f.name);
 }
 
-const char *key()
-{
-       return ASTERISK_GPL_KEY;
-}
+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
+);