bugBattle for Wesnoth - Bugs: bug #21964, assertion failure in mp connect...

 
 
Show feedback again

bug #21964: assertion failure in mp connect engine, when starting LoW 3rd scenario

Submitted by:  Chris Beck <involution>
Submitted on:  Wed 23 Apr 2014 10:11:42 PM UTC  
 
Category: BugSeverity: 5 - Blocker
Priority: 5 - NormalItem Group:  None of the others
Status: FixedPrivacy: Public
Assigned to: Andrius Silinskas <thunderstruck>Open/Closed: Closed
Release: 1.11.13Operating System: Linux Mint 16

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon 28 Apr 2014 11:12:03 PM UTC, comment #1:

I started trying to debug this:

It looks like in side_engine::update_controller_options, it is possible that if the controller lock settings are a certain way, then the controller_options_ vector will be empty, causing assertion failure in side_engine::update_current_controller_index.

Here's my branch I used for debugging, it just has some extra assertions and debugging output. https://github.com/cbeck88/wesnoth/tree/debug_21964
I have found that in side_engine::update_controller_options, the asserting that controller_options_.size()>0 will fail in the buggy test case.

thunderstruck, is it alright if I assign this to you?

Chris Beck <involution>
Project Member
Wed 23 Apr 2014 10:11:42 PM UTC, original submission:

Immediately after winning scenario 2 "The Hostile Mountains", the host will crash with this assertion failure. It seems to happen whether you win by reaching the goal post or killing the orcs.

wesnoth: src/multiplayer_connect_engine.cpp:1262: void mp::side_engine::update_current_controller_index(): Assertion `current_controller_index_ < controller_options_.size()' failed.

I also tested after reverting the server controller tweak commits and it didn't seem to help.

5e046acb2541f59f7a27eb02f6033a82d552aad4 server controller tweaks
6b8e303ee26c3935da864d25ce88146345a6a92f fixup server controller tweaks
4e2762fe6cab47a0dc1b076915a8078059bef8b2 fixup scenario transitions (add controller tweaks to load next scenario)

Chris Beck <involution>
Project Member

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by shadowmaster (Updated the item)
  • -unavailable- added by thunderstruck (Updated the item)
  • -unavailable- added by involution (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 28 May 2014 10:43:53 PM UTCshadowmasterOpen/ClosedOpen=>Closed
    Sat 10 May 2014 01:34:00 PM UTCthunderstruckStatusNone=>Fixed
    Mon 28 Apr 2014 11:12:03 PM UTCinvolutionAssigned toNone=>thunderstruck
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup