]> git.sesse.net Git - ffmpeg/blobdiff - libavcodec/pnm.c
add "memory" to the clobber list we change memory so we need it, this also fixes...
[ffmpeg] / libavcodec / pnm.c
index 30a65701699f42a9c54d689806943c083fe172d5..b729c4c529a670b4a83d3a0fb9f85acd7cb4050e 100644 (file)
@@ -2,19 +2,21 @@
  * PNM image format
  * Copyright (c) 2002, 2003 Fabrice Bellard.
  *
- * This library is free software; you can redistribute it and/or
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
+ * version 2.1 of the License, or (at your option) any later version.
  *
- * This library is distributed in the hope that it will be useful,
+ * FFmpeg is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 #include "avcodec.h"
 #include "mpegvideo.h" //only for ParseContext
@@ -26,16 +28,16 @@ typedef struct PNMContext {
     AVFrame picture;
 } PNMContext;
 
-static inline int pnm_space(int c)  
+static inline int pnm_space(int c)
 {
     return (c == ' ' || c == '\n' || c == '\r' || c == '\t');
 }
 
-static void pnm_get(PNMContext *sc, char *str, int buf_size) 
+static void pnm_get(PNMContext *sc, char *str, int buf_size)
 {
     char *s;
     int c;
-    
+
     /* skip spaces and comments */
     for(;;) {
         c = *sc->bytestream++;
@@ -47,7 +49,7 @@ static void pnm_get(PNMContext *sc, char *str, int buf_size)
             break;
         }
     }
-    
+
     s = str;
     while (sc->bytestream < sc->bytestream_end && !pnm_space(c)) {
         if ((s - str)  < buf_size - 1)
@@ -74,7 +76,7 @@ static int pnm_decode_header(AVCodecContext *avctx, PNMContext * const s){
     if (!strcmp(buf1, "P4")) {
         avctx->pix_fmt = PIX_FMT_MONOWHITE;
     } else if (!strcmp(buf1, "P5")) {
-        if (avctx->codec_id == CODEC_ID_PGMYUV) 
+        if (avctx->codec_id == CODEC_ID_PGMYUV)
             avctx->pix_fmt = PIX_FMT_YUV420P;
         else
             avctx->pix_fmt = PIX_FMT_GRAY8;
@@ -111,13 +113,13 @@ static int pnm_decode_header(AVCodecContext *avctx, PNMContext * const s){
         /* check that all tags are present */
         if (w <= 0 || h <= 0 || maxval <= 0 || depth <= 0 || tuple_type[0] == '\0' || avcodec_check_dimensions(avctx, w, h))
             return -1;
-                   
+
         avctx->width = w;
         avctx->height = h;
         if (depth == 1) {
             if (maxval == 1)
                 avctx->pix_fmt = PIX_FMT_MONOWHITE;
-            else 
+            else
                 avctx->pix_fmt = PIX_FMT_GRAY8;
         } else if (depth == 3) {
             avctx->pix_fmt = PIX_FMT_RGB24;
@@ -155,7 +157,7 @@ static int pnm_decode_header(AVCodecContext *avctx, PNMContext * const s){
     return 0;
 }
 
-static int pnm_decode_frame(AVCodecContext *avctx, 
+static int pnm_decode_frame(AVCodecContext *avctx,
                         void *data, int *data_size,
                         uint8_t *buf, int buf_size)
 {
@@ -168,10 +170,10 @@ static int pnm_decode_frame(AVCodecContext *avctx,
     s->bytestream_start=
     s->bytestream= buf;
     s->bytestream_end= buf + buf_size;
-    
+
     if(pnm_decode_header(avctx, s) < 0)
         return -1;
-    
+
     if(p->data[0])
         avctx->release_buffer(avctx, p);
 
@@ -182,7 +184,7 @@ static int pnm_decode_frame(AVCodecContext *avctx,
     }
     p->pict_type= FF_I_TYPE;
     p->key_frame= 1;
-    
+
     switch(avctx->pix_fmt) {
     default:
         return -1;
@@ -198,6 +200,8 @@ static int pnm_decode_frame(AVCodecContext *avctx,
     do_read:
         ptr = p->data[0];
         linesize = p->linesize[0];
+        if(s->bytestream + n*avctx->height > s->bytestream_end)
+            return -1;
         for(i = 0; i < avctx->height; i++) {
             memcpy(ptr, s->bytestream, n);
             s->bytestream += n;
@@ -211,6 +215,8 @@ static int pnm_decode_frame(AVCodecContext *avctx,
             n = avctx->width;
             ptr = p->data[0];
             linesize = p->linesize[0];
+            if(s->bytestream + n*avctx->height*3/2 > s->bytestream_end)
+                return -1;
             for(i = 0; i < avctx->height; i++) {
                 memcpy(ptr, s->bytestream, n);
                 s->bytestream += n;
@@ -233,6 +239,8 @@ static int pnm_decode_frame(AVCodecContext *avctx,
     case PIX_FMT_RGBA32:
         ptr = p->data[0];
         linesize = p->linesize[0];
+        if(s->bytestream + avctx->width*avctx->height*4 > s->bytestream_end)
+            return -1;
         for(i = 0; i < avctx->height; i++) {
             int j, r, g, b, a;
 
@@ -268,7 +276,7 @@ static int pnm_encode_frame(AVCodecContext *avctx, unsigned char *outbuf, int bu
     *p = *pict;
     p->pict_type= FF_I_TYPE;
     p->key_frame= 1;
-    
+
     s->bytestream_start=
     s->bytestream= outbuf;
     s->bytestream_end= outbuf+buf_size;
@@ -296,12 +304,12 @@ static int pnm_encode_frame(AVCodecContext *avctx, unsigned char *outbuf, int bu
     default:
         return -1;
     }
-    snprintf(s->bytestream, s->bytestream_end - s->bytestream, 
+    snprintf(s->bytestream, s->bytestream_end - s->bytestream,
              "P%c\n%d %d\n",
              c, avctx->width, h1);
     s->bytestream += strlen(s->bytestream);
     if (avctx->pix_fmt != PIX_FMT_MONOWHITE) {
-        snprintf(s->bytestream, s->bytestream_end - s->bytestream, 
+        snprintf(s->bytestream, s->bytestream_end - s->bytestream,
                  "%d\n", 255);
         s->bytestream += strlen(s->bytestream);
     }
@@ -313,7 +321,7 @@ static int pnm_encode_frame(AVCodecContext *avctx, unsigned char *outbuf, int bu
         s->bytestream += n;
         ptr += linesize;
     }
-    
+
     if (avctx->pix_fmt == PIX_FMT_YUV420P) {
         h >>= 1;
         n >>= 1;
@@ -347,7 +355,7 @@ static int pam_encode_frame(AVCodecContext *avctx, unsigned char *outbuf, int bu
     *p = *pict;
     p->pict_type= FF_I_TYPE;
     p->key_frame= 1;
-    
+
     s->bytestream_start=
     s->bytestream= outbuf;
     s->bytestream_end= outbuf+buf_size;
@@ -382,14 +390,14 @@ static int pam_encode_frame(AVCodecContext *avctx, unsigned char *outbuf, int bu
     default:
         return -1;
     }
-    snprintf(s->bytestream, s->bytestream_end - s->bytestream, 
+    snprintf(s->bytestream, s->bytestream_end - s->bytestream,
              "P7\nWIDTH %d\nHEIGHT %d\nDEPTH %d\nMAXVAL %d\nTUPLETYPE %s\nENDHDR\n",
              w, h, depth, maxval, tuple_type);
     s->bytestream += strlen(s->bytestream);
-    
+
     ptr = p->data[0];
     linesize = p->linesize[0];
-    
+
     if (avctx->pix_fmt == PIX_FMT_RGBA32) {
         int j;
         unsigned int v;
@@ -448,9 +456,10 @@ static int pam_probe(AVProbeData *pd)
 }
 #endif
 
+#ifdef CONFIG_PNM_PARSER
 static int pnm_parse(AVCodecParserContext *s,
                            AVCodecContext *avctx,
-                           uint8_t **poutbuf, int *poutbuf_size, 
+                           uint8_t **poutbuf, int *poutbuf_size,
                            const uint8_t *buf, int buf_size)
 {
     ParseContext *pc = s->priv_data;
@@ -467,8 +476,8 @@ retry:
         pnmctx.bytestream_end= pc->buffer + pc->index;
     }else{
         pnmctx.bytestream_start=
-        pnmctx.bytestream= buf;
-        pnmctx.bytestream_end= buf + buf_size;
+        pnmctx.bytestream= (uint8_t *) buf; /* casts avoid warnings */
+        pnmctx.bytestream_end= (uint8_t *) buf + buf_size;
     }
     if(pnm_decode_header(avctx, &pnmctx) < 0){
         if(pnmctx.bytestream < pnmctx.bytestream_end){
@@ -491,14 +500,14 @@ retry:
 #endif
         next= END_NOT_FOUND;
     }else{
-        next= pnmctx.bytestream - pnmctx.bytestream_start 
+        next= pnmctx.bytestream - pnmctx.bytestream_start
             + avpicture_get_size(avctx->pix_fmt, avctx->width, avctx->height);
         if(pnmctx.bytestream_start!=buf)
             next-= pc->index;
         if(next > buf_size)
             next= END_NOT_FOUND;
     }
-    
+
     if(ff_combine_frame(pc, next, (uint8_t **)&buf, &buf_size)<0){
         *poutbuf = NULL;
         *poutbuf_size = 0;
@@ -516,6 +525,7 @@ AVCodecParser pnm_parser = {
     pnm_parse,
     ff_parse_close,
 };
+#endif /* CONFIG_PNM_PARSER */
 
 #ifdef CONFIG_PGM_ENCODER
 AVCodec pgm_encoder = {
@@ -527,7 +537,7 @@ AVCodec pgm_encoder = {
     pnm_encode_frame,
     NULL, //encode_end,
     pnm_decode_frame,
-    .pix_fmts= (enum PixelFormat[]){PIX_FMT_GRAY8, -1}, 
+    .pix_fmts= (enum PixelFormat[]){PIX_FMT_GRAY8, -1},
 };
 #endif // CONFIG_PGM_ENCODER
 
@@ -541,7 +551,7 @@ AVCodec pgmyuv_encoder = {
     pnm_encode_frame,
     NULL, //encode_end,
     pnm_decode_frame,
-    .pix_fmts= (enum PixelFormat[]){PIX_FMT_YUV420P, -1}, 
+    .pix_fmts= (enum PixelFormat[]){PIX_FMT_YUV420P, -1},
 };
 #endif // CONFIG_PGMYUV_ENCODER
 
@@ -555,7 +565,7 @@ AVCodec ppm_encoder = {
     pnm_encode_frame,
     NULL, //encode_end,
     pnm_decode_frame,
-    .pix_fmts= (enum PixelFormat[]){PIX_FMT_RGB24, -1}, 
+    .pix_fmts= (enum PixelFormat[]){PIX_FMT_RGB24, -1},
 };
 #endif // CONFIG_PPM_ENCODER
 
@@ -569,7 +579,7 @@ AVCodec pbm_encoder = {
     pnm_encode_frame,
     NULL, //encode_end,
     pnm_decode_frame,
-    .pix_fmts= (enum PixelFormat[]){PIX_FMT_MONOWHITE, -1}, 
+    .pix_fmts= (enum PixelFormat[]){PIX_FMT_MONOWHITE, -1},
 };
 #endif // CONFIG_PBM_ENCODER
 
@@ -583,6 +593,6 @@ AVCodec pam_encoder = {
     pam_encode_frame,
     NULL, //encode_end,
     pnm_decode_frame,
-    .pix_fmts= (enum PixelFormat[]){PIX_FMT_RGB24, PIX_FMT_RGBA32, PIX_FMT_GRAY8, PIX_FMT_MONOWHITE, -1}, 
+    .pix_fmts= (enum PixelFormat[]){PIX_FMT_RGB24, PIX_FMT_RGBA32, PIX_FMT_GRAY8, PIX_FMT_MONOWHITE, -1},
 };
 #endif // CONFIG_PAM_ENCODER