Zandronum Chat on our Discord Server Get the latest version: 3.2.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
0000849Zandronum[All Projects] Bugpublic2012-05-17 10:012018-09-30 19:52
ReporterDusk 
Assigned ToDusk 
PriorityhighSeveritycrashReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version1.0-beta 
Target VersionFixed in Version1.0 
Summary0000849: Stronghold STR20 triggers an assertion failure on server
DescriptionWhen a client connects to a Stronghold server on STR20, the server will instantly trigger an assertion failure.
Steps To ReproduceStart up a server with strnghld_v1.pk3 and change to STR20. Connect a client and the server will in most cases abort instantly, in other cases you may need to do an additional "map str20" to trigger it.
Additional Informationcompatflags changed to: 0
Connect (v0.98e-alpha): 127.0.0.1:10667
skulltag: /home/dusk/src/skulltag/src/p_sight.cpp:647: bool P_CheckSight(const AActor*, const AActor*, int): Assertion `t2 != __null' failed.
Aborted
Attached Files

- Relationships
related to 0000995closedTorr Samaho Server crash with Brutal Doom 

-  Notes
User avatar (0003653)
Dusk (developer)
2012-05-23 00:52

 Fixed:'https://bitbucket.org/CrimsonDusk/notebola/changeset/ee687a1e2fa7 [^]'

There was a mispositioned player check that was done after P_CheckSight, not before it, allowing NULL to be passed to P_CheckSight, triggering the assert fail.
User avatar (0003657)
Torr Samaho (administrator)
2012-05-27 06:42

 I added your patch. It should have no effect on any of the official binaries (other than pr_newchasedir() being called less often) though, since assert should do nothing in release mode binaries. That's why I imported your patch directly and didn't update the history txt.
User avatar (0004118)
Dusk (developer)
2012-07-29 00:14

 Didn't know that one about assertions.. anyway it doesn't happen with the latest beta build anymore (was compiled from source fwiw) and thus I'd consider it fixed.

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-05-17 10:01 Dusk New Issue
2012-05-17 14:00 Dusk Description Updated View Revisions
2012-05-17 14:00 Dusk Steps to Reproduce Updated View Revisions
2012-05-22 23:54 Dusk Assigned To => Dusk
2012-05-22 23:54 Dusk Status new => assigned
2012-05-23 00:52 Dusk Note Added: 0003653
2012-05-23 00:52 Dusk Status assigned => needs testing
2012-05-23 00:52 Dusk Status needs testing => needs review
2012-05-27 06:42 Torr Samaho Note Added: 0003657
2012-05-27 06:42 Torr Samaho Status needs review => needs testing
2012-06-09 13:22 Torr Samaho Category General => Bug
2012-07-29 00:14 Dusk Note Added: 0004118
2012-07-29 00:14 Dusk Status needs testing => resolved
2012-07-29 00:14 Dusk Fixed in Version => 1.0-beta
2012-07-29 00:14 Dusk Resolution open => fixed
2012-08-23 01:54 Blzut3 Fixed in Version 1.0-beta => 1.0
2012-08-28 11:13 Dusk Relationship added related to 0000995
2018-09-30 19:52 Blzut3 Status resolved => closed




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2025 MantisBT Team
Powered by Mantis Bugtracker