엊그제 1.8.30 보안패치 올렸다고 한동안 안해도 되겠지~ 하고 있으면 큰코다쳐요.
[수정]
엊그제 1.8.31 보안패치 올렸다고 한숨 돌리고 있어도 곤란합니다!
오늘자 1.8.32까지 업데이트하셔야 합니다!
[수정]
중요 보안패치가 또 나왔습니다... 1.8.33...
[수정]
중요 보안패치가 또 나왔습니다... 1.8.34...
수정된 부분도 몇 줄 안 되니, 당장 업데이트가 힘들다면 패치된 부분(member.controller.php)만이라도 적용하세요.
10년 전 XE 초창기 버전부터 1.8.301.8.33까지 모든 버전이 취약합니다.
라이믹스도 1.8.311.8.34 패치까지 적용된 버전을 master 브랜치에 올렸으니 업데이트하시기 바랍니다.
업데이트하다가 에러나서 몇 시간 동안 사이트 먹통되는 것이 업데이트 안 하는 것보다 나아요.
그만큼 심각한 보안취약점입니다.
이런 게 왜 이제서야 발견된 것이여... ㅠㅠ
@기진곰
해당 패치를 잠깐 봤는데, 임시글 삭제하는 procMemberDeleteSavedDocument 함수에서
deleteDocument 처리 전에 본인의 글인지 확인을 안 하는게 문제라는 건데 (물론 임시글 여부인지 확인하는 코드도 포함되어있지만 )
( document_srl 값을 강제 변동하는 경우에 대한 우려인듯한데 )
어차피 deleteDocument 에 가면, 삭제전에 $oDocument->isGranted() 로 삭제권한여부를 다시 체크하기 때문에 실제적으로는 문제가 없지 않나요? ( isGranted 는 본인글 또는 관리자 를 제외하고는 안 되니)
( 물론 deleteDocument before 트리거에는 영향을 줄 수 있긴하겠지만.. 거의 무관할테고 )
패치를 해두면 훨씬 해당 함수에 의의는 맞지만.. 패치를 안 해도 뚫리진 않을듯 보여서요.???
?
보안이슈로 패치 안 되어있는 파일 살펴보면
https://github.com/xpressengine/xe-core/commit/ffa67bf0cc94e44973c0a427a5052f770328332c
member_srl 확인과 status 가 temp 인지 확인하는 문구가 추가되었다고만 나와있네요
전 이것만 보고 판단해서 몰랐는는데..
말씀해주신걸 보고, 혹시나 해서 기존 XE 버전들과 비교해보니
이 패치의 핵심은..
$oDocumentController->deleteDocument($document_srl,true); 가
$oDocumentController->deleteDocument($document_srl); 로 바뀐부분인데.. ( 위에건 더 효율을 높이기 위해 추가된 부분이고 )
정작 이 이슈쪽에는 원본도
$oDocumentController->deleteDocument($document_srl); 로 되어있던 상태에서 패치된거여서
그냥 이 이슈만 보고 수정하는 분들은 빼먹겠네요...
확인해보니 Xe 1.8.30 에서도 $oDocumentController->deleteDocument($document_srl,true); 로 되어있네요. XE 1.8.31 에서 true 가 빠진거고...
제가 github 를 잘 몰라서 그런데
혹시 https://github.com/xpressengine/xe-core/commit/ffa67bf0cc94e44973c0a427a5052f770328332c 를 직접 변경 가능하면
기존 원본 쪽을 $oDocumentController->deleteDocument($document_srl,true); 로 바꿔놔서 이 부분이 변경된거라는 걸 꼭 알 수 있게 해줄 수 있었으면 하네요
