From 50c42776e76e8d4ab3cdabbf441f4bfb034b1957 Mon Sep 17 00:00:00 2001 From: chicpro Date: Fri, 24 Oct 2014 11:01:34 +0900 Subject: [PATCH] =?UTF-8?q?SQL=20Injection=20=EB=B0=8F=20=EA=B4=80?= =?UTF-8?q?=EB=A6=AC=EC=9E=90=EA=B0=80=20=EA=B2=8C=EC=8B=9C=EA=B8=80=20?= =?UTF-8?q?=EC=88=98=EC=A0=95=EB=95=8C=20=EC=A0=95=EB=B3=B4=20=EB=B0=98?= =?UTF-8?q?=EC=98=81=EB=90=98=EB=8F=84=EB=A1=9D=20=EC=88=98=EC=A0=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- bbs/write.php | 10 +++++----- bbs/write_update.php | 18 ++++++++++++++---- 2 files changed, 19 insertions(+), 9 deletions(-) diff --git a/bbs/write.php b/bbs/write.php index 4a8e9672f..6235a4adc 100644 --- a/bbs/write.php +++ b/bbs/write.php @@ -294,10 +294,10 @@ $homepage = ""; if ($w == "" || $w == "r") { if ($is_member) { if (isset($write['wr_name'])) { - $name = get_text(cut_str($write['wr_name'],20)); + $name = get_text(cut_str(stripslashes($write['wr_name']),20)); } - $email = $member['mb_email']; - $homepage = get_text($member['mb_homepage']); + $email = get_email_address($member['mb_email']); + $homepage = get_text(stripslashes($member['mb_homepage'])); } } @@ -318,9 +318,9 @@ if ($w == '') { } } - $name = get_text(cut_str($write['wr_name'],20)); + $name = get_text(cut_str(stripslashes($write['wr_name']),20)); $email = get_email_address($write['wr_email']); - $homepage = get_text($write['wr_homepage']); + $homepage = get_text(stripslashes($write['wr_homepage'])); for ($i=1; $i<=G5_LINK_COUNT; $i++) { $write['wr_link'.$i] = get_text($write['wr_link'.$i]); diff --git a/bbs/write_update.php b/bbs/write_update.php index 857889f90..b6abef8c3 100644 --- a/bbs/write_update.php +++ b/bbs/write_update.php @@ -412,14 +412,24 @@ if ($w == '' || $w == 'r') { $wr_homepage = addslashes(clean_xss_tags($member['mb_homepage'])); } else { $mb_id = $wr['mb_id']; - $wr_name = $wr['wr_name']; - $wr_email = $wr['wr_email']; - $wr_homepage = $wr['wr_homepage']; + if(isset($_POST['wr_name']) && $_POST['wr_name']) + $wr_name = clean_xss_tags(trim($_POST['wr_name'])); + else + $wr_name = addslashes(clean_xss_tags($wr['wr_name'])); + if(isset($_POST['wr_email']) && $_POST['wr_email']) + $wr_email = get_email_address(trim($_POST['wr_email'])); + else + $wr_email = addslashes($wr['wr_email']); + if(isset($_POST['wr_homepage']) && $_POST['wr_homepage']) + $wr_homepage = addslashes(clean_xss_tags($_POST['wr_homepage'])); + else + $wr_homepage = addslashes(clean_xss_tags($wr['wr_homepage'])); } } else { $mb_id = ""; // 비회원의 경우 이름이 누락되는 경우가 있음 - //if (!trim($wr_name)) alert("이름은 필히 입력하셔야 합니다."); + if (!trim($wr_name)) alert("이름은 필히 입력하셔야 합니다."); + $wr_name = clean_xss_tags(trim($_POST['wr_name'])); $wr_email = get_email_address(trim($_POST['wr_email'])); }