The code conditions put inside the function is_multimedia_file are
reverse to the name i.e, we need to negate the return to actually
check if the file is a multimedia file. So, change the code and usage
path to align both the name and comparision conditions.
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Amit Sahrawat <a.sahrawat@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
int ret;
if (sublen > slen)
int ret;
if (sublen > slen)
ret = memcmp(s + slen - sublen, sub, sublen);
if (ret) { /* compare upper case */
ret = memcmp(s + slen - sublen, sub, sublen);
if (ret) { /* compare upper case */
char upper_sub[8];
for (i = 0; i < sublen && i < sizeof(upper_sub); i++)
upper_sub[i] = toupper(sub[i]);
char upper_sub[8];
for (i = 0; i < sublen && i < sizeof(upper_sub); i++)
upper_sub[i] = toupper(sub[i]);
- return memcmp(s + slen - sublen, upper_sub, sublen);
+ return !memcmp(s + slen - sublen, upper_sub, sublen);
int count = le32_to_cpu(sbi->raw_super->extension_count);
for (i = 0; i < count; i++) {
int count = le32_to_cpu(sbi->raw_super->extension_count);
for (i = 0; i < count; i++) {
- if (!is_multimedia_file(name, extlist[i])) {
+ if (is_multimedia_file(name, extlist[i])) {
set_cold_file(inode);
break;
}
set_cold_file(inode);
break;
}