View Issue Details

IDProjectCategoryView StatusLast Update
0001725Spring engineGeneralpublic2009-11-13 12:14
ReporterSirMaverick Assigned ToAuswaschbar  
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionfixed 
Product Version0.80.5 
Summary0001725: long "MyPlayerName" hangs Spring
DescriptionIf you set MyPlayerName in script.txt to a very long name, the game does not start. It just shows "Waiting for server to start".
Additional Informatione.g.
MyPlayerName=unnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamedunnamed;
TagsNo tags attached.
Attached Files
script1.txt (Attachment missing)
script2.txt (Attachment missing)
Checked infolog.txt for Errors

Activities

hoijui

2009-11-12 12:46

reporter   ~0004309

i was not able to reproduce this. just when i forgot to set the Name in the [PLAYERx] section, i got a similar symptom.
i tried with the name you gave under Additional Information (around 750 chars), and the game started just fine.
GCC debug build on Linux, running in GDB

SirMaverick

2009-11-12 14:47

reporter   ~0004311

Both scripts show this behavior for me.

SirMaverick

2009-11-12 15:07

reporter   ~0004312

> GCC debug build on Linux, running in GDB

Hangs for me too.

Kloot

2009-11-12 15:29

developer   ~0004313

For me those scripts just trigger an assertion in debug mode; the Name field of [Player1] needs to match MyPlayerName in [Game]. After setting player 1's name to unnamedunnamedunnamed... as well Springs starts normally.

hoijui

2009-11-12 16:00

reporter   ~0004314

ok, so we found out that it is not about the length of the name already now, just that they differ. one thing we could do is hacing a nicer error message in such a case, and exiting spring. but we can/will never protect agaisnt all errors that could happen when a user manually edits script.txt, so i see it as questionable to be worth doing.
what do you want, SirMaverick?

SirMaverick

2009-11-12 16:55

reporter   ~0004315

With debug build I get "not found in script, rejecting connection attempt". But no assertion triggered.

I think the input should be verified and give some message in case of errors. I get no message when the names are different in release build.

Auswaschbar

2009-11-13 12:14

reporter   ~0004320

Spring now quits and give a nice "Unknwon palyer" message when this happens

Issue History

Date Modified Username Field Change
2009-11-11 15:58 SirMaverick New Issue
2009-11-12 12:46 hoijui Note Added: 0004309
2009-11-12 14:45 SirMaverick File Added: script1.txt
2009-11-12 14:46 SirMaverick File Added: script2.txt
2009-11-12 14:47 SirMaverick Note Added: 0004311
2009-11-12 15:07 SirMaverick Note Added: 0004312
2009-11-12 15:29 Kloot Note Added: 0004313
2009-11-12 16:00 hoijui Note Added: 0004314
2009-11-12 16:55 SirMaverick Note Added: 0004315
2009-11-13 12:14 Auswaschbar Note Added: 0004320
2009-11-13 12:14 Auswaschbar Status new => resolved
2009-11-13 12:14 Auswaschbar Resolution open => fixed
2009-11-13 12:14 Auswaschbar Assigned To => Auswaschbar