fix(batch): EgovFlatFileByteReader 공유 가변 static LINE_CRLF 동시성·데이터 손상 제거

[z3rotig4r]

문제

EgovFlatFileByteReader.LINE_CRLF는 public static int로 선언된 공유 가변 전역인데, 인스턴스 setter setOsType()이 이 전역을 직접 덮어쓴다. 이 단일 전역 값이 reader의 고정길이 버퍼 크기(new byte[length + LINE_CRLF], reader.read(..., length + LINE_CRLF))와 EgovFixedByteTokenizer의 CRLF 차감(byteString.length - LINE_CRLF)에 동시에 쓰인다.

osType이 서로 다른 두 reader 빈(UNIX·WINDOWS)이 한 컨텍스트에 공존하면, 마지막 setOsType 호출이 전역을 덮어쓴다(last-writer-wins). 그 결과 다른 빈의 고정길이 필드 경계가 1바이트씩 밀려 무성(silent) 데이터 손상으로 이어진다. CWE-362(경합 조건)·CWE-366(공유 변수 경합)·CWE-471에 해당한다.

수정

전역 가변 static을 인스턴스 상태로 옮긴다.

• reader: public static int LINE_CRLF 제거 → private int lineCrlf(기본값 WINDOWS=2). 버퍼 사이징은 인스턴스 필드를 참조하고 getLineCrlf()를 추가한다.
• tokenizer: reader의 static 참조를 끊고 자체 private int lineCrlf(기본값 2)와 setLineCrlf(int)·setOsType(String)를 둔다.

빈마다 값을 독립으로 보유하므로 상호 오염이 사라진다.

동작 보존

단일 reader의 기본/WINDOWS 구성은 값(2)이 그대로라 출력 바이트가 동일하다. CRLF를 차감하지 않는 형제 클래스 EgovFixedByteLengthTokenizer는 손대지 않았다.

주의(후방호환)

기존에는 reader에 setOsType("UNIX")를 한 번 호출하면 공유 static을 통해 tokenizer의 차감값도 1로 전파됐다. 분리 이후에는 UNIX 고정길이 잡의 경우 tokenizer에도 setOsType("UNIX")(또는 setLineCrlf(1))를 명시 설정해야 한다. reader만 UNIX로 두면 tokenizer가 기본값 2로 차감해 IncorrectLineLengthException이 발생한다. 또한 public static LINE_CRLF 필드가 제거되므로, 외부에서 이 필드를 직접 참조하던 코드가 있다면 컴파일에 영향을 준다.

테스트

신규 4건을 추가했다.

• reader A(UNIX)·B(WINDOWS) 인스턴스 격리 — 한 빈의 설정이 다른 빈에 영향을 주지 않음을 확인하는 회귀 가드
• 기본값 2 보존
• tokenizer 기본(WINDOWS) 토큰 경계 보존
• tokenizer UNIX 경로

mvn -pl Batch/org.egovframe.rte.bat.core -am test 실행, 해당 모듈 테스트 18건 모두 통과.