Author Topic: Crash with latest trunk (5712)  (Read 2022 times)

0 Members and 1 Guest are viewing this topic.

Crash with latest trunk (5712)
Whilst testing out the BP2 intro movie ( :D ) with the latest trunk, it crashes with the following error:
Code: [Select]
Unknown error.

In sexpression: ( when
   ( true )
   ( set-camera-position 1011 361 17394 )
   ( set-camera-target
      "Nelson"
      "Fighterbay"
   )
)
(Error appears to be: Fighterbay)
ntdll.dll! ZwWaitForSingleObject + 21 bytes
kernel32.dll! WaitForSingleObjectEx + 67 bytes
kernel32.dll! WaitForSingleObject + 18 bytes
fs2_open_3_6_11d.exe! SCP_DumpStack + 294 bytes
fs2_open_3_6_11d.exe! Error + 229 bytes
fs2_open_3_6_11d.exe! post_process_mission + 1577 bytes
fs2_open_3_6_11d.exe! parse_mission + 1531 bytes
fs2_open_3_6_11d.exe! parse_main + 449 bytes
fs2_open_3_6_11d.exe! mission_load + 215 bytes
fs2_open_3_6_11d.exe! game_start_mission + 189 bytes
fs2_open_3_6_11d.exe! game_enter_state + 482 bytes
fs2_open_3_6_11d.exe! gameseq_set_state + 310 bytes
fs2_open_3_6_11d.exe! game_process_event + 242 bytes
fs2_open_3_6_11d.exe! gameseq_process_events + 152 bytes
fs2_open_3_6_11d.exe! game_main + 745 bytes
fs2_open_3_6_11d.exe! WinMain + 330 bytes
fs2_open_3_6_11d.exe! __tmainCRTStartup + 358 bytes
fs2_open_3_6_11d.exe! WinMainCRTStartup + 15 bytes
kernel32.dll! BaseThreadInitThunk + 18 bytes
ntdll.dll! RtlInitializeExceptionChain + 99 bytes
ntdll.dll! RtlInitializeExceptionChain + 54 bytes

It's pulling the UNKNOWN_ERROR value from line 1772 of sexp.cpp while checking the sexps in post_process_mission.
How it's getting there, I don't know :P
I haven't quite figured out how to make a much smaller case, but if I do figure out how, I'll post it here.
STRONGTEA. Why can't the x86 be sane?

 

Offline Zacam

  • Magnificent Bastard
  • Administrator
  • 211
  • I go Sledge-O-Matic on Spammers
    • Steam
    • Twitter
    • ModDB Feature
Re: Crash with latest trunk (5712)
Take
Code: [Select]
return SEXP_CHECK_UNKNOWN_ERROR; // just in case of something going wrong - it won't trickle down
and replace it with:
Code: [Select]
break;

Don't bother putting the SEXP_CHECK back in.
Report MediaVP issues, now on the MediaVP Mantis! Read all about it Here!
Talk with the community on Discord
"If you can keep a level head in all this confusion, you just don't understand the situation"

¤[D+¬>

[08/01 16:53:11] <sigtau> EveningTea: I have decided that I am a 32-bit registerkin.  Pronouns are eax, ebx, ecx, edx.
[08/01 16:53:31] <EveningTea> dhauidahh
[08/01 16:53:32] <EveningTea> sak
[08/01 16:53:40] * EveningTea froths at the mouth
[08/01 16:53:40] <sigtau> i broke him, boys

 
Re: Crash with latest trunk (5712)
Mantis'd: #2066
STRONGTEA. Why can't the x86 be sane?

 

Offline Goober5000

  • HLP Loremaster
  • 214
    • Goober5000 Productions
Re: Crash with latest trunk (5712)
Take
Code: [Select]
return SEXP_CHECK_UNKNOWN_ERROR; // just in case of something going wrong - it won't trickle down
and replace it with:
Code: [Select]
break;

Don't bother putting the SEXP_CHECK back in.
:wtf:

I thought you said that the return statement was throwing an unreachable code warning?

 

Offline Zacam

  • Magnificent Bastard
  • Administrator
  • 211
  • I go Sledge-O-Matic on Spammers
    • Steam
    • Twitter
    • ModDB Feature
Re: Crash with latest trunk (5712)
It is, as conditions for it's check no longer exist. Hence changing the line back to the break; it originally was.

But that has already been done via commit 5713. Just adding this for thread closure.

Also, for those worrying about the Test Build I released here, while it is based on 5712 it does not have the reported problem. I fixed the issue a little differently than goober did (in fact, 5713 watches what I did) so that is a working 5712-based build.
« Last Edit: December 14, 2009, 02:25:26 am by Zacam »
Report MediaVP issues, now on the MediaVP Mantis! Read all about it Here!
Talk with the community on Discord
"If you can keep a level head in all this confusion, you just don't understand the situation"

¤[D+¬>

[08/01 16:53:11] <sigtau> EveningTea: I have decided that I am a 32-bit registerkin.  Pronouns are eax, ebx, ecx, edx.
[08/01 16:53:31] <EveningTea> dhauidahh
[08/01 16:53:32] <EveningTea> sak
[08/01 16:53:40] * EveningTea froths at the mouth
[08/01 16:53:40] <sigtau> i broke him, boys

  

Offline The E

  • He's Ebeneezer Goode
  • Moderator
  • 213
  • Nothing personal, just tech support.
    • Steam
    • Twitter
Re: Crash with latest trunk (5712)
I think I'll close this, then.
If I'm just aching this can't go on
I came from chasing dreams to feel alone
There must be changes, miss to feel strong
I really need lifе to touch me
--Evergrey, Where August Mourns