diff options
author | František Dvořák <valtri@users.sourceforge.net> | 2006-04-05 22:12:17 +0000 |
---|---|---|
committer | František Dvořák <valtri@users.sourceforge.net> | 2006-04-05 22:12:17 +0000 |
commit | 639bd610bfcb1a028b91a5c3d125045c8de6d8b7 (patch) | |
tree | 6247df0c3a36b4a8eb11e0a764ff3e99632a619b /src/demuxers/id3.c | |
parent | 22924ef0924502e0d9b1652651d40270602ee7a0 (diff) | |
download | xine-lib-639bd610bfcb1a028b91a5c3d125045c8de6d8b7.tar.gz xine-lib-639bd610bfcb1a028b91a5c3d125045c8de6d8b7.tar.bz2 |
1) various warning fixes
2) [coverity] fixes:
- 279 (w32): test should be on input parametr type, not typeW
- 267 (aout): can be ignored but there was warning anyway
- 234 (w32): missing free on error & exit
- 152 (w32): missing free on error & exit
- 148 (w32): fixed bound of static array in test
- 101 (ffdec): defense code when avodec_encode_video will fail (just defense to the future,
should never happen with current ffmpeg in xine)
- 98, 99 (w32): check negative handle by _x_assert (it's used as index to array)
- 90 (w32): negate pointer check (+added check for size)
- 89 (w32): added NULL and size check
CVS patchset: 7959
CVS date: 2006/04/05 22:12:17
Diffstat (limited to 'src/demuxers/id3.c')
-rw-r--r-- | src/demuxers/id3.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/demuxers/id3.c b/src/demuxers/id3.c index 29cb3d4c4..ffa022a59 100644 --- a/src/demuxers/id3.c +++ b/src/demuxers/id3.c @@ -29,7 +29,7 @@ * * ID3v2 specs: http://www.id3.org/ * - * $Id: id3.c,v 1.11 2005/09/16 23:01:10 tmattern Exp $ + * $Id: id3.c,v 1.12 2006/04/05 22:12:18 valtri Exp $ */ #ifdef HAVE_CONFIG_H @@ -227,11 +227,13 @@ static int id3v2_parse_genre(char* dest, char *src, int len) { return 1; } +#if 0 /* parse an unsynchronized 16bits integer */ static uint16_t BE_16_synchsafe(uint8_t buf[2]) { return ((uint16_t)(buf[0] & 0x7F) << 7) | (uint16_t)(buf[1] & 0x7F); } +#endif /* parse an unsynchronized 24bits integer */ static uint32_t BE_24_synchsafe(uint8_t buf[3]) { |