웹취약점 항목 : 널 포인터 역참조에 대해서 여쭙고 싶습니다

Question

&#64;RequestMapping(value &#61; &#34;/SBExcelLoad.do&#34;, method &#61; RequestMethod.POST)protected void SBExcelLoad(HttpServletRequest request, HttpServletResponse response) throws Exception {MultipartRequest multi &#61; null;ServletContext context &#61; request.getSession().getServletContext();String realPath &#61; context.getRealPath(&#34;/&#34;);File f &#61; new File(realPath &#43; &#34;kr\softbowl&#34;);if (!f.isDirectory()) {if (f.mkdirs()) {LOG.debug(&#34;Directory Make !&#34;);}}JSONArray arrGridData &#61; new JSONArray();JSONParser JSONdataParser &#61; new JSONParser();try {String strFileInputName &#61; multi.getParameter(&#34;SBHE___SB_FileInputName&#34;);웹취약점 보안 사항에 대해서 해당소스 밑줄친 부분에 이런답변을 받았는데요multi 변수 선언에서 null이 배정된 채로 사용하게되면 세크멘테이션 폴트나 널 포인터 예외가 발생할 수 있습니다.getParameter(..) 함수 호출 전 null 검사를 하거나 예외처리를 하세요.상단에 MultipartRequest multi &#61; null; 부분을 널체크를....?어떻게 바꾸라는 의도일까요 ...?혹시 이런걸 바라는걸까요 ...?try { String strFileInputName &#61; multi.getParameter(&#34;SBHE___SB_FileInputName&#34;); if (multi &#61;&#61; null) { throw new IllegalArgumentException(&#34;String cannot be null&#34;); //웹취약점 보완 null pointer 역참조 }

카카오콘 · Answer

이미 본문에 해결 방법까지 명시 되어 있는 것으로 보입니다.getParameter(..) 함수 호출 전 null 검사를 하거나 예외처리를 하세요.String strFileInputName = multi.getParameter("SBHE___SB_FileInputName");이 명령을 실행하기 전, null 체크를 수행하라는 말 같은데 다른 문제가 있으신지요?

오후 · Answer

String strFileInputName &#61; multi.getParameter(&#34;SBHE___SB_FileInputName&#34;); 파라매터 : SBHE___SB_FileInputName 가 널인지 체크 or strFileInputName 를 널인지 체크후 사용 하라는 내용입니다.

출처불명 · Answer

multi가 null로 선언만 되어있고 인스턴스가 없으니인스턴스 만들어라 이 얘기 아닌가요..?

roggue2 · Answer

if(multi.getParameter("SBHE___SB_FileInputName") == null){throw new IllegalArgumentException("String cannot be null"); //웹취약점 보완 null pointer 역참조}String strFileInputName = multi.getParameter("SBHE___SB_FileInputName");순서상 이렇게 처리하는게 맞을 듯 합니다.

웹취약점 항목 : 널 포인터 역참조에 대해서 여쭙고 싶습니다

4개의 답변

기술 목록