[💬] 마비노기 돈복사버그 터지는거 보고 느끼는거

[퍅셔미(wvvwvwwvw)]

유니티는 구릴지 몰라도.. C#은 사랑한다...

해설 : 이번에 마비노기에서 돈복사버그 크게 터졌는데, 구식ui호환패치 하는 과정에서 아이템 구매가격 오버플로우가 발생한게 원인임. 마비노기 플레이오네 엔진은 C++이고, C++은 오버플로우 발생시 Exception을 발생시키지않음. 즉, C++은 개발자의 실수로인한 문제발생을 원천차단할 수 있는 기능을 언어단에서 많이 지원하지 않음.

로우레벨 언어와 하이레벨 언어를 비교할때, 성능상의 이점과 휴먼에러방지의 이점을 저울질을 하게되는데 이번사례가 대표적인 예시가 되는 것 같음.

[211214(tomatoss)]

[해당 댓글은 삭제되었습니다.]

해당 댓글은 삭제되었습니다.

[퍅셔미(wvvwvwwvw)]

개발할땐 휴먼에러 잡아주고 il2cpp로 성능잡아주면 토끼 두마리 다잡았네

[2호기(wlsrb014756)]

[dualSkewer(jmm4658)]

마소가 작정하고 키운 아들내미

[파베르(jaba001)]

프로그래밍 바벨탑이 생겨야한다

[소닉쫄따구테일즈(dcz3xv7zgu14)]

웃기네, 왜 클라에서 금액 오버플로우가 발생했는데, 그게 서버에 반영이 되지. 설마 금액계산을 서버에서 안하고 클라를 신뢰하게 코딩한건가... 미쳤네

[퍅셔미(wvvwvwwvw)]

마비노기 금화가 단순히 숫자 오르락내리락이 아니라 금화주머니나 아이템 칸을 차지하는 또다른 아이템 형태로 돼있어서 클라에서 일부 처리하도록 초창기에 구현이 잡혀있었나봄? 아무리 그래도 클라 값을 신뢰하는 미친 구현방식으로 지금까지 운영하진 못했을거같고, 클라에서 인벤토리시뮬+금화값시뮬 한 값이랑 서버에서 금화값만시뮬 한 값이랑 대조하는식으로 검증했을거임. 금화값 시뮬은 완전 서버클라 완전 대칭으로 짰을거고. 그래서 아마 구ui 신ui가 각각 대응되는 서버코드도 다르겠지..ㅋㅋㅋㅋㅋ 뭐 뇌피셜이긴하지만

[퍅셔미(wvvwvwwvw)]

신ui는 서버 클라 양측모두 큰 정수값에대한 대응이 잘 돼있는데, 구 ui는 큰 정수값에 대응이안돼있는 형태로 추정됨. 그래서 오버플로 발생->이사단이 난거

[클라리네이(ggs7839)]

나도 처음 이 기사 봤을때 왜 클라를 신뢰한거지?라고 생각하긴 했는데, 그래도 그 정도는 아닐것 같고, 클라나 서버나 그냥 같은 자료형으로 짠거 아닐까? 서버에서 만들려는 수량을 클라에서 받고, 단일 금액 곱하기해서 자료형 오버플로우가 서버에서도 난거 아닐까 싶긴하다.

[소닉쫄따구테일즈(dcz3xv7zgu14)]

ㅇㅇ 위에형들 말대로 왠지 클라랑 서버 둘다 대칭으로 같은자료형으로 짰는게 맞을듯, 보통 컨텐츠 단위로 개발자한테 일시키면, 클라 서버 둘다 작업하는경우가 많긴해서... 신입개발자 뽑아서 시켰나보네 ㅋ

[oㅇO0(niece9310)]

서버가 c++인거 아닐까

[ㅇㅇ 1(211.54)]

그러면 ui랑 상관있는게 이상하지 근데 공고보니 마비노기 서버 cpp맞는듯?

[ㅇㅇ 2(222.238)]

유저가 클라에서 상점으로 구매 판매 이런걸 하면 서버쪽에 패킷이든뭐든 데이터 보내서 그거보고 디비에넣고 유저한테 데이터 보내고 이런식인데 거기에 오버플로우 대책을 안해놓은거인가보다

[퍅셔미(wvvwvwwvw)]

근데 오버플로 대책안해서 터지는거 심심할때마다 보이네, 메이저 회사들도 진짜 많이들 안하는구나

[익명(qpalzm0147)]

솔플겜에선 치트키를 쓰면 본인에게 손해

[MeroZemory(first5916)]

고신뢰성 소프트웨어를 c++로 만드려면 checked arithmetic 사용하는게 좋음 좀 더 구체적으로는 사용할 원시 타입 각각에 대해 래핑 클래스 만들고 연산자 오버로딩하고 overflow처럼 문법 적으로 검출 안되는 산술 오류 런타임에 잡아주면 되는데, 이런걸 다 해놓은 라이브러리들이 있음. 한 예로 마소의 SafeInt Library