待辦事項 #43870

Client returns with EXIT_SUCCESS after failing to load tileset

啟用日期: 2022-02-15 18:33 最後更新: 2022-03-03 01:10

回報者:
負責人:
類型:
狀態:
關閉
元件:
里程碑:
優先權:
5 - 中
嚴重程度:
5 - 中
處理結果:
修正
檔案:
3

細節

Client is supposed to return EXIT_FAILURE when it aborts due to failed tileset load:

--

log_error(_("Can't load requested tileset %s!"), forced_tileset_name);
client_exit();
return EXIT_FAILURE;

--

But client_exit() there never returns (should be marked noreturn, but that's another issue) to do that 'return EXIT_FAILURE'. Instead it does 'exit(EXIT_SUCCESS);'

Ticket History (3/9 Histories)

2022-02-15 18:33 Updated by: cazfi
  • New Ticket "Client returns with EXIT_SUCCESS after failing to load tileset" created
2022-02-20 09:26 Updated by: cazfi
  • 負責人 Update from (無) to cazfi
  • 處理結果 Update from to Accepted
  • 里程碑 Update from (無) to 3.0.1 (closed)
2022-02-20 10:07 Updated by: cazfi
評語

Going to push also to S2_6 (S3_0 patch applies) as this helps debugging efforts when porting to new environments (and that's what S2_6 is all about now - only used for platforms for which S3_0 is not easy to reach)

2022-02-20 21:14 Updated by: cazfi
評語

Reply To cazfi

Going to push also to S2_6 (S3_0 patch applies)

S2_6 will need patch of its own - xaw-client has calls to client_exit() (which is probably the reason the function is not static in client_main.c - something I wondered when making the patch for later branches)

2022-02-20 21:26 Updated by: cazfi
評語

Reply To cazfi

S2_6 will need patch of its own

Attached

2022-03-03 01:10 Updated by: cazfi
  • 狀態 Update from 開啟 to 關閉
  • 處理結果 Update from Accepted to 修正

編輯

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » 登入