Zandronum Chat @ irc.zandronum.com
#zandronum
Get the latest version: 2.1.2
Source Code

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0003016Zandronum[All Projects] Bugpublic2017-02-17 02:062017-04-27 01:44
ReporterStrikerMan780 
Assigned ToTorr Samaho 
PriorityhighSeverityblockReproducibilityalways
StatusresolvedResolutionfixed 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version3.0-beta 
Target Version3.0Fixed in Version3.0 
Summary0003016: GAMEINFO auto-loaded PWADs load twice if joining through Doomseeker or Doom Explorer
DescriptionIf a PWAD contains a GAMEINFO lump, and requests other wads get loaded with it, they will get loaded twice if someone joins through Doomseeker or Doom Explorer.
Steps To ReproduceGet skulltag_data_fixed.pk3:http://www.wad-archive.com/wad/048122af1c22cb7b146ad76a20bd5dca [^]

Put it in the same folder as Zandronum.exe

Then, get the attached GAMEINFO.PK3, stick it in there too.

Launch a server with zandronum.exe -host -file GAMEINFO.PK3

It'll automatically pull in skulltag_data_fixed.pk3.

Now, join the server with Doom Explorer or Doomseeker. The WAD will be loaded twice, and the game will crash.
Attached Files? file icon GAMEINFO.PK3 [^] (282 bytes) 2017-02-17 02:06

- Relationships

-  Notes
User avatar (0016871)
StrikerMan780 (reporter)
2017-02-17 02:08
edited on: 2017-02-17 02:09

I suggest making Zandronum not pull WADs/PK3s from the LOAD line of GAMEINFO if said WAD/PK3 was already loaded via the command line.

User avatar (0016915)
Torr Samaho (administrator)
2017-02-27 13:26

I can easily flag the files loaded by GAMEINFO as automatically loaded, so that they won't appear on the list of pwads reported to the launcher. Unfortunately, this way the launcher doesn't know that the client needs the wad to join.

Quote from StrikerMan780

I suggest making Zandronum not pull WADs/PK3s from the LOAD line of GAMEINFO if said WAD/PK3 was already loaded via the command line.

This sounds like something GZDoom should do. Does the latest GZDoom already do this by any chance? Nevertheless, won't this change the order in which the files are loaded?
User avatar (0016916)
StrikerMan780 (reporter)
2017-02-27 17:13
edited on: 2017-02-27 17:22

EDIT: Just realized it wouldn't change the load order, because Zandronum already sends the loaded wads to the master server in the same order the server does. (They always load just before the WAD with the GAMEINFO lump.)

Also, GZDoom has this issue as well, just tested it. It will double-load files.

User avatar (0016926)
Torr Samaho (administrator)
2017-03-03 07:00

I thought a bit more about this and think the best would be to simply have the client (in the sense of a binary started with "-connect") not load the wads specified in GAMEINFO. The server already added those in the correct order to the list of files advertised to the launcher, so the launcher will add all necessary files to the command that starts the client.
User avatar (0016931)
Torr Samaho (administrator)
2017-03-05 14:55

Should be fixed now using the approach from 0003016:0016926.
User avatar (0017210)
Combinebobnt (reporter)
2017-04-19 01:27

r170416-0710 it worked
User avatar (0017211)
StrikerMan780 (reporter)
2017-04-19 01:35

Nice. Thank you.

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2017-02-17 02:06 StrikerMan780 New Issue
2017-02-17 02:06 StrikerMan780 File Added: GAMEINFO.PK3
2017-02-17 02:08 StrikerMan780 Note Added: 0016871
2017-02-17 02:09 StrikerMan780 Note Edited: 0016871 View Revisions
2017-02-17 02:09 StrikerMan780 Note Edited: 0016871 View Revisions
2017-02-27 13:26 Torr Samaho Note Added: 0016915
2017-02-27 13:27 Torr Samaho Status new => feedback
2017-02-27 17:13 StrikerMan780 Note Added: 0016916
2017-02-27 17:13 StrikerMan780 Status feedback => new
2017-02-27 17:17 StrikerMan780 Note Edited: 0016916 View Revisions
2017-02-27 17:18 StrikerMan780 Note Edited: 0016916 View Revisions
2017-02-27 17:18 StrikerMan780 Note Edited: 0016916 View Revisions
2017-02-27 17:19 StrikerMan780 Note Edited: 0016916 View Revisions
2017-02-27 17:22 StrikerMan780 Note Edited: 0016916 View Revisions
2017-03-03 07:00 Torr Samaho Note Added: 0016926
2017-03-03 07:01 Torr Samaho Assigned To => Torr Samaho
2017-03-03 07:01 Torr Samaho Status new => feedback
2017-03-05 14:55 Torr Samaho Note Added: 0016931
2017-03-05 14:56 Torr Samaho Status feedback => needs testing
2017-03-05 14:56 Torr Samaho Target Version => 3.0
2017-04-19 01:27 Combinebobnt Note Added: 0017210
2017-04-19 01:35 StrikerMan780 Note Added: 0017211
2017-04-27 01:44 Ru5tK1ng Status needs testing => resolved
2017-04-27 01:44 Ru5tK1ng Resolution open => fixed
2017-04-27 01:44 Ru5tK1ng Fixed in Version => 3.0






Questions or other issues? Contact Us.

Links


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker