Do proper bounds checking in formats (bug #1356)
[asterisk/asterisk.git] / formats / format_g729.c
index f4ba3db..68dde21 100755 (executable)
@@ -32,7 +32,7 @@
 #include <machine/endian.h>
 #endif
 
-/* Some Ideas for this code came from makeg729e.c by Jeffery Chilton */
+/* Some Ideas for this code came from makeg729e.c by Jeffrey Chilton */
 
 /* Portions of the conversion code are by guido@sienanet.it */
 
@@ -150,12 +150,12 @@ static int g729_write(struct ast_filestream *fs, struct ast_frame *f)
                ast_log(LOG_WARNING, "Asked to write non-G729 frame (%d)!\n", f->subclass);
                return -1;
        }
-       if (f->datalen % 20) {
-               ast_log(LOG_WARNING, "Invalid data length, %d, should be multiple of 20\n", f->datalen);
+       if (f->datalen % 10) {
+               ast_log(LOG_WARNING, "Invalid data length, %d, should be multiple of 10\n", f->datalen);
                return -1;
        }
        if ((res = write(fs->fd, f->data, f->datalen)) != f->datalen) {
-                       ast_log(LOG_WARNING, "Bad write (%d/20): %s\n", res, strerror(errno));
+                       ast_log(LOG_WARNING, "Bad write (%d/10): %s\n", res, strerror(errno));
                        return -1;
        }
        return 0;
@@ -183,8 +183,9 @@ static int g729_seek(struct ast_filestream *fs, long sample_offset, int whence)
                offset = max - bytes;
        if (whence != SEEK_FORCECUR) {
                offset = (offset > max)?max:offset;
-               offset = (offset < min)?min:offset;
        }
+       // protect against seeking beyond begining.
+       offset = (offset < min)?min:offset;
        if (lseek(fs->fd, offset, SEEK_SET) < 0)
                return -1;
        return 0;