fix: security issue with image upload (#175)
This commit is contained in:
parent
33a39c7042
commit
7b5e166e28
@ -65,7 +65,8 @@
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
function resizeAndUploadLogo($uploadedFile, $uploadDir, $name) { $targetWidth = 70;
|
function resizeAndUploadLogo($uploadedFile, $uploadDir, $name) {
|
||||||
|
$targetWidth = 70;
|
||||||
$targetHeight = 48;
|
$targetHeight = 48;
|
||||||
|
|
||||||
$timestamp = time();
|
$timestamp = time();
|
||||||
@ -154,6 +155,15 @@
|
|||||||
$icon = getLogoFromUrl($iconUrl, '../../images/uploads/logos/', $name);
|
$icon = getLogoFromUrl($iconUrl, '../../images/uploads/logos/', $name);
|
||||||
} else {
|
} else {
|
||||||
if (!empty($_FILES['paymenticon']['name'])) {
|
if (!empty($_FILES['paymenticon']['name'])) {
|
||||||
|
$fileType = mime_content_type($_FILES['paymenticon']['tmp_name']);
|
||||||
|
if (strpos($fileType, 'image') === false) {
|
||||||
|
$response = [
|
||||||
|
"success" => false,
|
||||||
|
"errorMessage" => translate('fill_all_fields', $i18n)
|
||||||
|
];
|
||||||
|
echo json_encode($response);
|
||||||
|
exit();
|
||||||
|
}
|
||||||
$icon = resizeAndUploadLogo($_FILES['paymenticon'], '../../images/uploads/logos/', $name);
|
$icon = resizeAndUploadLogo($_FILES['paymenticon'], '../../images/uploads/logos/', $name);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -157,6 +157,11 @@
|
|||||||
$logo = getLogoFromUrl($logoUrl, '../../images/uploads/logos/', $name);
|
$logo = getLogoFromUrl($logoUrl, '../../images/uploads/logos/', $name);
|
||||||
} else {
|
} else {
|
||||||
if (!empty($_FILES['logo']['name'])) {
|
if (!empty($_FILES['logo']['name'])) {
|
||||||
|
$fileType = mime_content_type($_FILES['logo']['tmp_name']);
|
||||||
|
if (strpos($fileType, 'image') === false) {
|
||||||
|
echo translate("fill_all_fields", $i18n);
|
||||||
|
exit();
|
||||||
|
}
|
||||||
$logo = resizeAndUploadLogo($_FILES['logo'], '../../images/uploads/logos/', $name);
|
$logo = resizeAndUploadLogo($_FILES['logo'], '../../images/uploads/logos/', $name);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user