MantisBT - VCMI
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0002455VCMIAI - Adventure Mappublic2016-08-12 07:292022-04-12 10:47
ReporterSXX 
Assigned ToSXX 
PrioritynormalSeveritycrashReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version0.98g 
Target VersionFixed in Version0.99 
Summary0002455: VCAI: Freeze on attempt to move artifact locks
DescriptionAI trying to move artifact locks and freeze in process.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0002458assigned Warmonger Artifacts from log might have id of ArtifactID::ART_LOCK (145) 
related to 0002461resolved SXX VCAI: Infinite loop when attempt to compare and exchange artifacts 
Attached Fileszip AIstuck_moveArtifactLocks.zip (1,299,944) 2016-08-12 07:29
https://bugs.vcmi.eu/file_download.php?file_id=2514&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-08-12 07:29SXXNew Issue
2016-08-12 07:29SXXStatusnew => assigned
2016-08-12 07:29SXXAssigned To => SXX
2016-08-12 07:29SXXFile Added: AIstuck_moveArtifactLocks.zip
2016-08-15 06:44WarmongerSeveritymajor => crash
2016-08-18 02:43SXXNote Added: 0006565
2016-08-18 04:00SXXNote Added: 0006566
2016-08-18 04:01SXXRelationship addedparent of 0002458
2016-08-18 04:01SXXRelationship deletedparent of 0002458
2016-08-18 04:01SXXRelationship addedrelated to 0002458
2016-08-18 04:01SXXStatusassigned => resolved
2016-08-18 04:01SXXFixed in Version => 0.99
2016-08-18 04:01SXXResolutionopen => fixed
2016-08-23 03:49SXXRelationship addedrelated to 0002461
2022-04-12 10:47PovelitelStatusresolved => closed

Notes
(0006565)
SXX   
2016-08-18 02:43   
Okay I find out why is this occur probably. Problem is that artifacts that loaded from mods might use id of 145 which actually mean ArtifactID::ART_LOCK.

So server see this and think AI trying to move lock...
(0006566)
SXX   
2016-08-18 04:00   
So here is the fix:

https://github.com/vcmi/vcmi/commit/39fe9472b60d05401aa2889d4698bac2c74a9c6b [^]

Problem with id conflict still need attention so I added separate bug.