Frame related code cleanup

Change-Id: Ic9a410660cf7ae9a349b811ece339df23e7a5fc9
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Version up(0.6.87 -> 0.6.88)

Change-Id: Iedbe0d4ef65d7bd5013c916dbf181b08c89a9ba5

Remove unnecessary define

Change-Id: Ifbd4d89cfa0e14e29b4d5e688cd07834899195b1
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

format IMELODY cleanup

Change-Id: I581d81b7b4b7043adadbf9759535721ba39c4712
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Format MMF cleanup

Change-Id: I59f5636051b3c0606204e00859e0c7388c63e2c6
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Format MIDI cleanup

Change-Id: I14f8bc26ebe25f94841e9b6fe9b444e89ba0b386
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove unused files

Change-Id: I2df45ccb64285f8fffac9543efdaa51adae37d11
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Format WAV cleanup

Change-Id: I9d22133e4dc7f49bb4cbc227cf56ab9132b35345
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove test code for directory support

Change-Id: I176b6b188bacc38591a1c8ee7257d045eb13b87d
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove memtrace

Change-Id: I674cc520cada75203dbdf58f3b2ab42e0d4f9bd5
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove format dummy

Change-Id: I1105b369d120b12051091cfa74ce0e9c9963879e
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Format AMR cleanup

Change-Id: If3f3a99f621f9119a22ca7a05758708aaab8cd63
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Format AAC cleanup

Change-Id: I41aff76fbc3fe6232ced2a086c8cebd0688355ab
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Internal functions cleanup

Change-Id: I6383006f55209ac298bbf6cffacba371a8d497ac
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Change image buf type

Change-Id: I0dcebf05e1517855785d6dc8f8f019844eb06657
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove timestamp

Change-Id: I3c98f7da11f5a460333f3403e96d0dbfc5f37b15
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove codec related code

'bCompressed' is false always.
So, codec related code are never called.

Change-Id: I014e6ee44e317d8fc575d2b750ba40b558a6a526
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove EXPORT_API

Change-Id: I9fcbab71b378ae58d2fa434dcffc3349c9bed8e6
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Improve allocation related code

Change-Id: I1c296b0d36420c47edf27ae232f4026b26f70292
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

function in mm_file_util_string cleanup

Change-Id: I2e9fe3ad4113dec5feb2ec98f56157c8b8db0a25
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Change return value

Change-Id: If166672ee78e765b2b1831de1c3aa9f3ab4b9cd2
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Use macro instead

Change-Id: Icc7875387e0cd21eebca93b6b8ced1414e2c47e2
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Check file before dlopen

Change-Id: Ie5691150dc0ee7bd76d138a417726ad20b1af307
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove duplicated condition for checking the frame length

Change-Id: I1be9feb05291cbd12719d8b7fef6270caf35a88c

Fix crash issue due to garbage value

  - If the result of g_convert() is empty string, the written_len is 0.
    Then (unsigned int)'written_len - 1' became garbage value and
    checking carriage return make crash.

(gdb)bt full
mmfile_string_convert ...mm_file_util_string.c:189
        i = 1395696
        result = 0xae45a410 ""
        err = 0x0
        written_len = 0

Change-Id: I61b4c3e4a6b938ce549844e163dc46f47398aa98

Unify duplicated codes regarding getting file size

Change-Id: I357be69f82a03b9e8ac6d7ba94cc9341d40a4e0a

Unify duplicate codes regarding content attribute

Change-Id: I9bc54faeb45eed7ea86275d5ceb990f61781b5fe

Fix typo

Change-Id: Ifac06fd27ff239f23444b99fff7bef1407b71d10

Replace return value to define instead of hardcoded number

Change-Id: Ia58d2d95a717fc962ef32f6e1c70698f60c35bb8

define enum for ID3tag to make codes simple

Change-Id: If2ebbfd36565d8878ad0f11da1e18742db4373d6

Bump to 0.6.85

Change-Id: Ia88c259d3110e21eb206cadb5951af5f3bf0590e

Remove unused code and apply coding rule

Change-Id: I13287f489082df64e03fb06208b60b588a3f4d5c

Reinforce comment and '__AvGetLastID3v2Offset' function

Change-Id: I8fc9af5c7003715aa8968a56203047a251d68c10

[code cleanup] Add to check hard return value for file_read

Change-Id: I329980a09b6be133d374376a6bbe7b833afcf271

[code cleanup] Remove duplicated buffer allocation & file read
and remove unnecessary code

Change-Id: I57ec5316e4bd26d14d83ded18007bb5d209c33cb

Seperate getting mp3 information into getting mp3_tag_info and mp3_stream_info

  - When 'mmfile_format_open' function is called, it read mp3 file
    to get mp3 information. Because scanning mp3 file call the function
    2~3 times, there was a loss in performance.
    So we changed the function call from open to read and
    seperated into two sub_function.

Change-Id: Ica77bed739c89e41e3643889e29ada6eadb7fd28

[code cleanup] Improved code readability for getting ID3Tag

Change-Id: I1be3ab6b95a7e0c75e349b6de365769ef8e3944e

Unify duplicated ID3v2header getting code

Change-Id: Ib875c1e2421632dfb69f864df037a9e2f82d6258

Remove redundant condition check code

Change-Id: I774056be4f064d0a6cc0f5160aba6f5cacc12fd9

Remove useless parameter v2numOfFrames

Change-Id: I6967a9637c370a884247104b93a89ff8d2272b73

Unify duplicated ID3tag extracting code

Change-Id: Idde219a2231733c90e70cd55c90d6b6f9325ab86

Remove id3 version specific 'mmfile_string_convert_v224' function and unreachable code

Change-Id: Ib528a6811d8197b51bb990e655e07a06062e4b4f

Remove duplicated code and unused parameter pre_checked

Change-Id: I6118f7dabce399f4fd731ac4bede1fef4462de50

Remove unused parameter pAuthor

Change-Id: I0ce1a7838763f26576c048035322752c835f221e

Remove unreachable code

 - Fix svace WGID 425086

 Varaible 'pForamtCtx' can't be null because of
 'NULL == formatContext->privateFormatData' checker.

Change-Id: I6e18ffad655bb3de10a8379257e65d473226dbb3

Update test code for macro refactoring

Change-Id: I4bc0fd5b84d0f97fdf924722477bb505cd7bdeb5

Fix coverity issue

  - Argument cannot be negative

  'ftell' returns signed long and it can be negative(-1).
  Unsigned int 'file_size(size)' can get tainted value.
  So signed long for ftell has been added and check if it is below zero.

Change-Id: Ibe651c19465acc57b2e4e31b705644887c18a623

Fix coverity issue

  - Resource leak
  Variable 'err_attr_name' should be freed by caller, but it is not used and freed.
  So, remove unused(useless) 'err_attr_name'.

Change-Id: I201c216828a05a0e620799145960de925c9ce524

Fix wrong duration for mpeg-ts

 Generally, mpegts has wrong start time(positive large value).
 So duration do not use start time for mpeg-ts.

Change-Id: I97ddc31860bdeb489abe41f4fbc7e9b61d2572d5

Fix build error due to toolchain upgrade (gcc6->gcc9)

  error: output trucated before the last format character [-Werror=format-truncation=]

Change-Id: I9a0c7460e1afc81e46d14a4190c89a644943bf64

Remove unused functions

Change-Id: Ib7ccc1e9da5c9793d0b2169c502e3df82e79a947

Replace SAFE_STRLCPY to mmfile_strdup

Change-Id: I81a8687b35b6066a7c6454e0af8242ec357b4f20

Remove unused variables

 - descriptionLen
 - pDescription
 - pRating
 - bTagVer2Found
 - bGenreUTF16

Change-Id: Id0ae4c97533d7a466686eacaf35af919d157e9f1

Code Cleanup for duplicated code of getting text encoding type

 - Add new funtion to get text encoding type

Change-Id: I33d11fa4f59c52f25b20b221bfe02a8a0bb5231d

Code Cleanup for duplicated code of getting synclyric

Change-Id: I20ab23c9cb0b3ad99121397cc402695f7961ee12

Code Cleanup for 'break' and add function to count the number of streams

Change-Id: I48131561ecded342993344d7198915ed9b935a75

Change global variable '_mm_file_format_mimetype_table' to static

Change-Id: I66368de864551766a51133c30f4b35827a2167d2

Improve id3tag parsing for duplicated code(skip new line)

Change-Id: I4914f579dadb63713efc8c37cce61a73db5b3c32

Fix coverity issue (memory leak)

Change-Id: I2494d0d8543ab3dc4fa88cd4520e0bbd20c7397a

Remove unused header file (duplicated with mm_util.h)

Change-Id: I6fb8771a23f61f1ed7aaf38d6daacefab5776eb3

Bump version to 0.6.80

 - Since this version, TC test value change is necessary.

Change-Id: Icd42fc0f7c2cef10e8e134fa21e353bb8128226c

Improve 'mm_file_id3tag_parse_v224' function to reduce Cyclomatic Complexity

 - It needs below UTC changes.
  https://review.tizen.org/gerrit/#/c/test/tct/native/api/+/214107/

Change-Id: Ie9ab72aa8d663ff759d12f2219edfa07c32c2f4c

Update global variable to static

Change-Id: Iad7002eb2932136ead5609d767620cc704d8f005

Bump version to 0.6.79

Change-Id: I8e2d044f79fcb07933ec6637174ed6902863ce0d

Improve PIC/APIC parsing of ID3tag to reduce CC and DC

Change-Id: Ibb0edee2b18f6ee03863686a39da700d43e7c852

Improve functions to reduce Duplicate Code

Change-Id: Ifc5191726c11039a818ecda28aeba609c5c02042

Improve '__mmfile_get_frame' function to reduce Cyclomatic Complexity

Change-Id: Ia0335b309420605b39dadef3b63c114e94f2564d

Improve '_get_contents_info' function to reduce Cyclomatic Complexity

Change-Id: I9b86b5432904d9475bc28a4f7e18ad42a1e42ea5

[DF190903-00975] Add code to support mpeg audio version 2, layer 1

Change-Id: I86c355baa73bb57285764f02ed00b7de2593c537

Improve 'MMFileFormatIsValidXXX' function to reduce DC

Change-Id: Id9ba66477f0b8766db1b33efb8d09bbb805b3b51

Improve '__AvConvertIMelody2MIDI' function to reduce DC

Change-Id: Ic751574c6677a1520121f36560d7b6762d2c8928

Improve '_mmf_MalibNextChunk' function and remove unused code

 - change 'switch ~ case' to 'looping const table'
 - remove unused function
 - remove unuseful condition (null check)

Change-Id: I227f94c6da3f6563c2ff05d1815bfee004634fa7

Remove unneccessary null check

 - 'mm_attrs_set_string/data_by_name' can set null string/data
 - artwork & thumbnail should null check due to memcpy

Change-Id: I219c365925eb03ddaffdbefa96677c7eb956aac8

Add new function to add bracket at genre tag

Change-Id: I2f9c9da246d573c53e2f364fb750afbf52f4e020

Replace duplicate '__getMimeType' and 'getMimeType' to 'mmfile_util_get_mimetype'

Change-Id: Ie01859409d1b6044c093c2de47f1e4449cf7b2aa

Improve 'mmfile_format_read_tag_ffmpg' function

Change-Id: I2edf99b08d29631d0363e1ce3493004ad99bfa21

Update debug msg

Change-Id: I4ca482bb7afb313de76f79a4624c563c197c679d

Update pkg version

Change-Id: I0f336e2e2cb9c5eeab27469aa465974496e58b5f

Change strcasecmp() to g_ascii_strcasecmp() because strcasecmp is locale sensitive.

Change-Id: Iff51728241d047f0af6f4dc2addbeecd5f573899

Merge "Support TPOS (Part of a Set)" into tizen

Merge "Just add description for the smta" into tizen

Increase _RETRY_SEARCH_LIMIT to 250. some contents need search count this much

Change-Id: I6825cea3e9a0a2a895b2e13af78940889c602744

Support TPOS (Part of a Set)

The ‘Part of a set’ frame is a numeric string that describes which part of a set the audio came from.
This frame is used if the source described in the “TALB” frame is divided into several mediums,
e.g. a double CD. The value MAY be extended with a “/” character and a numeric string containing the total number of parts in the set. E.g. “1/2”.

Change-Id: Ic0613adda4b54834892e7592df34598b9d14be9b

Just add description for the smta

Change-Id: I661cd6fe8667e3716e1269574836af63a61ac263

Support smta value 0x02

Change-Id: I7dc1f10917a706e03447a97b5ef69d945064d664

Change the size and nmemb of fread() to check the return value

Change-Id: Icff42f9f037df9b44cd5ab07420dab1037747463

Add to check the return of fseek()

 - If fseek() is failed, -1 is returned and errno is set to indicate the errors.

Change-Id: I7174ca625da1026ebe07cb72fd97459d89c05d75

Add to check the file permission before mm_file access a file.

- error case
Metadata-extractor did not return Permission denied.

Change-Id: I5ef74a95dcca60e451d9c1e55aab67694f7f208e

Coverity issue fix

Change-Id: Id90eb786575e82376fd0535ea11fd0fadfcd5470

Fix format error in logs

Change-Id: I024c8d532c47c3a58aa13f4b6e5fa5c6a647d7a2

Fix build error without gtests

Change-Id: I7eb22626683c57234d272302228beb831093d434

Merge "Adding initial structure for unittest" into tizen

Adding initial structure for unittest

- It has only one testcases for libmedia_utils of unittest
- It would be enabled by gtest build with '--define "gtests 1"'
- Two test contents would be copyied to '/usr/bin' directory

Change-Id: I6988de63bfd3c2cea7aa8518bf83d700d09d38cd

Apply new attribute API of libmm-common

- Remove dependency of mm_attrs_private.h.
- Use new API set in mm_attrs.h instead of those.

Change-Id: Icd720cc02b677b881e4266c8a4a0cd79f72ab5ce
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

Remove dlog from Requires

Change-Id: Ie270508a787cc45064acfc7a3c5f1b17d658c8d2

Remove duplicate verification code

Use mmfile_free instead of FREE_EX
Remove duplicate null check

Change-Id: I73101a08098966b1e026d2d0c384632abc1c74d9
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

[PLM P180706-00145]Fix character set issue for some languages used in Europe. And svace issues

Change-Id: I90bdeb4d5ba89b0f130995bfe575766a24ce37bb

Add fortify, stack canary option

Change-Id: I52c4da8092619c3d0d165082bb8a9be64766fab0
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Implement Spherical video metadata V2 extraction in MKV/WebM

- No stereo-mode support added as far as it is stored using the
  existing StereoMode element specified in the Matroska spec. and
  can be supported in the Matroska format parser

- WebM files prepared with MKVMerge tool with "-w" option turned on
  (force WebM sub-format) are not recognized by libmm-fileinfo

- Ambisonic audio is not specified in Google's RFC v2

Change-Id: I4935602afb658bfa6971be247a4ffd1e3332fe55
Signed-off-by: Mykola Alieksieiev <m.alieksieie@samsung.com>

support webm format

Change-Id: I4154332ff696b0efc060ad58876ad4d892e1715c