Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001205Zandronum[All Projects] Bugpublic2012-12-08 20:472018-09-30 20:21
ReporterDusk 
Assigned ToDusk 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version1.1Fixed in Version1.1 
Summary0001205: A_CheckSight performs on the client-side on its own
DescriptionClients execute A_CheckSight on their own. If A_CheckSight's target state involves deleting the actor, the client will believe that the actor does not exist.

Furthermore, they are executing A_CheckSight when nothing should be executing anyway. In the attached example WAD, the Imp is replaced with a version that does A_CheckSight right after it spawns and goes into an idle loop afterwards. A fence blocking everything AND SIGHT rises up. A newcomer client has the imp execute A_CheckSight (which it already should not do). The imp then goes into the GoDie state where it is deleted.
Steps To Reproduce- Host a server with attached example wad and connect a client to it.
- Move to the second room and hit the switch. An imp will spawn atop a pad and the barrier will rise, blocking everything and sight.
- Connect a second client to the server. For this client, the imp has deleted itself and is not there.
- If sv_cheats is enabled, the second client can noclip into the imp room and observe the desync.
Attached Files? file icon ACheckSightTest.wad [^] (4,130 bytes) 2012-12-08 20:47

- Relationships

-  Notes
User avatar (0005706)
Dusk (developer)
2013-01-06 14:47

 Fixed this:'https://bitbucket.org/CrimsonDusk/neurosphere/commits/3800 [^]'
If this is OK I'll push it to a new head.
User avatar (0005707)
Torr Samaho (administrator)
2013-01-06 17:20

 Looks good.
User avatar (0005711)
Dusk (developer)
2013-01-06 22:41

 'https://bitbucket.org/CrimsonDusk/neurosphere/commits/3803 [^]'

Finally managed to make an automated script to do this pushing..
User avatar (0005719)
Torr Samaho (administrator)
2013-01-07 20:54

 Thanks! Pulled.
User avatar (0006070)
StrikerMan780 (reporter)
2013-02-24 13:24

 Is there a test build for this? In the publically released 1.1 build, A_CheckSight stopped working entirely where it seemed to work fine in 1.0...
User avatar (0006074)
Torr Samaho (administrator)
2013-02-24 19:04
edited on: 2013-02-24 19:05

 This should contain Dusk's fix, but so does the public beta build Zandronum 1.1-alpha-130120-0905.
User avatar (0006231)
Arco (updater)
2013-04-05 17:44
edited on: 2013-04-07 09:35

 V1.1-alpha-130317-1940 does not conform the fix.

Edit: This is fixed. It was simply a misinterpretation error on my part...
User avatar (0006258)
Torr Samaho (administrator)
2013-04-06 18:59

 Please report exactly what's happening and what you think should happen instead.
User avatar (0006264)
Qent (updater)
2013-04-07 01:10

 It works as expected for me.

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
2012-12-08 20:47 Dusk New Issue
2012-12-08 20:47 Dusk File Added: ACheckSightTest.wad
2013-01-06 14:47 Dusk Note Added: 0005706
2013-01-06 14:47 Dusk Assigned To => Dusk
2013-01-06 14:47 Dusk Status new => needs review
2013-01-06 17:20 Torr Samaho Note Added: 0005707
2013-01-06 22:41 Dusk Note Added: 0005711
2013-01-07 20:54 Torr Samaho Note Added: 0005719
2013-01-07 20:54 Torr Samaho Status needs review => needs testing
2013-01-07 20:54 Torr Samaho Target Version => 1.1
2013-02-24 13:24 StrikerMan780 Note Added: 0006070
2013-02-24 19:04 Torr Samaho Note Added: 0006074
2013-02-24 19:05 Torr Samaho Note Edited: 0006074 View Revisions
2013-04-05 17:44 Arco Note Added: 0006231
2013-04-05 19:48 Dusk Status needs testing => assigned
2013-04-06 18:59 Torr Samaho Note Added: 0006258
2013-04-07 01:10 Qent Note Added: 0006264
2013-04-07 09:35 Arco Note Edited: 0006231 View Revisions
2013-04-07 11:51 Dusk Status assigned => resolved
2013-04-07 11:51 Dusk Fixed in Version => 1.1
2013-04-07 11:51 Dusk Resolution open => fixed
2018-09-30 20:21 Blzut3 Status resolved => closed




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker