MantisBT - Zandronum
View Issue Details
0001205Zandronum[All Projects] Bugpublic2012-12-08 20:472018-09-30 20:21
Dusk 
Dusk 
normalminoralways
closedfixed 
 
1.11.1 
0001205: A_CheckSight performs on the client-side on its own
Clients 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.
- 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.
No tags attached.
? ACheckSightTest.wad (4,130) 2012-12-08 20:47
/tracker/file_download.php?file_id=898&type=bug
Issue History
2012-12-08 20:47DuskNew Issue
2012-12-08 20:47DuskFile Added: ACheckSightTest.wad
2013-01-06 14:47DuskNote Added: 0005706
2013-01-06 14:47DuskAssigned To => Dusk
2013-01-06 14:47DuskStatusnew => needs review
2013-01-06 17:20Torr SamahoNote Added: 0005707
2013-01-06 22:41DuskNote Added: 0005711
2013-01-07 20:54Torr SamahoNote Added: 0005719
2013-01-07 20:54Torr SamahoStatusneeds review => needs testing
2013-01-07 20:54Torr SamahoTarget Version => 1.1
2013-02-24 13:24StrikerMan780Note Added: 0006070
2013-02-24 19:04Torr SamahoNote Added: 0006074
2013-02-24 19:05Torr SamahoNote Edited: 0006074bug_revision_view_page.php?bugnote_id=6074#r3350
2013-04-05 17:44ArcoNote Added: 0006231
2013-04-05 19:48DuskStatusneeds testing => assigned
2013-04-06 18:59Torr SamahoNote Added: 0006258
2013-04-07 01:10QentNote Added: 0006264
2013-04-07 09:35ArcoNote Edited: 0006231bug_revision_view_page.php?bugnote_id=6231#r3460
2013-04-07 11:51DuskStatusassigned => resolved
2013-04-07 11:51DuskFixed in Version => 1.1
2013-04-07 11:51DuskResolutionopen => fixed
2018-09-30 20:21Blzut3Statusresolved => closed

Notes
(0005706)
Dusk   
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.
(0005707)
Torr Samaho   
2013-01-06 17:20   
Looks good.
(0005711)
Dusk   
2013-01-06 22:41   
'https://bitbucket.org/CrimsonDusk/neurosphere/commits/3803 [^]'

Finally managed to make an automated script to do this pushing..
(0005719)
Torr Samaho   
2013-01-07 20:54   
Thanks! Pulled.
(0006070)
StrikerMan780   
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...
(0006074)
Torr Samaho   
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.

(0006231)
Arco   
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...

(0006258)
Torr Samaho   
2013-04-06 18:59   
Please report exactly what's happening and what you think should happen instead.
(0006264)
Qent   
2013-04-07 01:10   
It works as expected for me.