Monkuar Posted March 6, 2012 Share Posted March 6, 2012 $ids = implode(",", $_POST['m']); $db->query('DELETE FROM friends WHERE friend_id IN ('.$db->escape($ids).') AND user_id = '.$pun_user['id'].'') or error('Unable to remove users from online list', __FILE__, __LINE__, $db->error()); redirect("s.php?section=Friends","Thanks, Friends removed "); Is there any other way to secure it better? I feel like im missing something, i just don't want to get hacked also added $ids = implode(",", $_POST['m']); if ($ids < 1) message("Incorrect Data"); Quote Link to comment https://forums.phpfreaks.com/topic/258378-how-to-i-secure-my-imploded-data/ Share on other sites More sharing options...
AyKay47 Posted March 6, 2012 Share Posted March 6, 2012 if $_POST['m'] contains an array of integers, the only sanitation you need to do is to intval the values to ensure they are integers. If you do that, there is no need to escape them. Quote Link to comment https://forums.phpfreaks.com/topic/258378-how-to-i-secure-my-imploded-data/#findComment-1324437 Share on other sites More sharing options...
Monkuar Posted March 6, 2012 Author Share Posted March 6, 2012 if $_POST['m'] contains an array of integers, the only sanitation you need to do is to intval the values to ensure they are integers. If you do that, there is no need to escape them. Would this work? $ids = implode(",", array_map('intval', $_POST['m'])); Quote Link to comment https://forums.phpfreaks.com/topic/258378-how-to-i-secure-my-imploded-data/#findComment-1324439 Share on other sites More sharing options...
scootstah Posted March 6, 2012 Share Posted March 6, 2012 if $_POST['m'] contains an array of integers, the only sanitation you need to do is to intval the values to ensure they are integers. If you do that, there is no need to escape them. Would this work? $ids = implode(",", array_map('intval', $_POST['m'])); Yes. I literally just posted that exact code but you already did. Quote Link to comment https://forums.phpfreaks.com/topic/258378-how-to-i-secure-my-imploded-data/#findComment-1324440 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.