The atom 'skip' should be handled by 'mov_read_leaf()', not by

'mov_read_default()'. This atom has the same behaviour than the atom 'free'.
patch by (Cedric VINCENT <c-vinc02 at bat710 dot univ-lyon1 dot fr>)
the second part of the patch hasnt been applied, but fixed slightly differently


git-svn-id: file:///var/local/repositories/ffmpeg/trunk@3276 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b

libavformat/mov.c

@@ -490,6 +490,7 @@ static int mov_read_hdlr(MOVContext *c, ByteIOContext *pb, MOV_atom_t atom)
 	    url_fskip(pb, len);
     }
 
+    url_fskip(pb, atom.size - (url_ftell(pb) - atom.offset));
     return 0;
 }
 
@@ -1444,7 +1445,7 @@ static const MOVParseTableEntry mov_default_parse_table[] = {
 { MKTAG( 'n', 'm', 'h', 'd' ), mov_read_leaf },
 { MKTAG( 'o', 'd', 'h', 'd' ), mov_read_default },
 { MKTAG( 's', 'd', 'h', 'd' ), mov_read_default },
-{ MKTAG( 's', 'k', 'i', 'p' ), mov_read_default },
+{ MKTAG( 's', 'k', 'i', 'p' ), mov_read_leaf },
 { MKTAG( 's', 'm', 'h', 'd' ), mov_read_leaf }, /* sound media info header */
 { MKTAG( 'S', 'M', 'I', ' ' ), mov_read_smi }, /* Sorrenson extension ??? */
 { MKTAG( 's', 't', 'b', 'l' ), mov_read_default },