Ticket #43870

Client returns with EXIT_SUCCESS after failing to load tileset

Open Date: 2022-02-15 18:33 Last Update: 2022-03-03 01:10

Reporter:
Owner:
Type:
Status:
Closed
Component:
MileStone:
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
Fixed
File:
3

Details

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
  • Owner Update from (None) to cazfi
  • Resolution Update from None to Accepted
  • Milestone Update from (None) to 3.0.1 (closed)
2022-02-20 10:07 Updated by: cazfi
Comment

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
Comment

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
Comment

Reply To cazfi

S2_6 will need patch of its own

Attached

2022-03-03 01:10 Updated by: cazfi
  • Status Update from Open to Closed
  • Resolution Update from Accepted to Fixed

Edit

Please login to add comment to this ticket » Login