[Freeciv-Dev] Re: Coredump with latest CVS

[Jeff Mallatt <jjm@xxxxxxxxxxxx>]

At 2000/02/03 16:32 , Jeff Mallatt wrote:
>So, I was poking around, and while looking at ai_manage_unit(), I noticed
>that after each sub-call (e.g., ai_manage_diplomat(), ai_manage_settler(),
>&c.) the code falls through to another branch of the if/else if/else
>if/else, which tests punit->type for something -- *but* punit may
>potentially be invalid upon return from any of the sub-calls, right?  So, I
>think we need to add 'return;'s after each ai_manage_xxx() call.

Here's a patch which adds these 'return;'s.

robust-ai_manage_unit.diff.gz
Description: Binary data

jjm