Anonymous | Login | Signup for a new account | 2021-03-05 21:42 UTC | ![]() |
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
0003726 | Zandronum | [All Projects] Bug | public | 2019-10-20 07:48 | 2020-05-05 22:47 | ||||||||
Reporter | eagle | ||||||||||||
Assigned To | Torr Samaho | ||||||||||||
Priority | high | Severity | crash | Reproducibility | always | ||||||||
Status | needs testing | Resolution | open | ||||||||||
Platform | Windows | OS | Windows Server 2012 R2 | OS Version | XP/Vista/7 | ||||||||
Product Version | 3.0 | ||||||||||||
Target Version | Fixed in Version | ||||||||||||
Summary | 0003726: Zandronum 3.0.1 - crash - Integer Overflow | ||||||||||||
Description | Zandronum 3.0.1 - crash - Integer Overflow, due to the calculation of decal coordinates in assembler assest. | ||||||||||||
Additional Information | I have my own correction of this error, if you notice my existence at all and react to this ticket in the next six years. | ||||||||||||
Attached Files | ![]() ![]() ![]() ![]() | ||||||||||||
![]() |
|
Torr Samaho (administrator) 2019-12-22 14:34 |
Quote from eagle I'm all ears. |
eagle (reporter) 2019-12-26 17:09 |
Remove the assembler insert in the function on which the crash occurred, the problem is because of it. |
Torr Samaho (administrator) 2020-01-12 20:53 |
Unfortunately, I can't get anything out of the crash log. Which function are you referring to? |
eagle (reporter) 2020-01-14 07:09 |
I uploaded a screenshot with the fix |
Torr Samaho (administrator) 2020-04-30 20:03 |
Thanks, I added your patch! |
eagle (reporter) 2020-05-05 06:45 |
this correction was not enough, so we went further... I attached a screenshot. |
Edward-san (developer) 2020-05-05 21:22 edited on: 2020-05-05 21:31 |
Can you attach a crash log obtained with the new build? Also, the new patch seems to change the scale code in the c code instead of the intended assembler code, contradicting the statement in the description that the assembler code was the culprit here. How did you discover this? Also, just to ask: is zandronum compiled by yourself, by any chance? |
eagle (reporter) 2020-05-05 21:58 |
I checked it on my own compilation, so my logs won't help here. and the problem there is huge values in variables obtained before division. Only the Assembly insert was replaced, and it is still called in many places in the render. |
eagle (reporter) 2020-05-05 21:59 |
This overflow error appears on huge maps when drawing decals. |
Edward-san (developer) 2020-05-05 22:29 edited on: 2020-05-05 22:34 |
Quote Indeed, from the code it looks like it is sufficient to have a decal on a very large wall. Can you attach an example wad reproducing the crash with this, which is also compatible with gzdoom? |
eagle (reporter) 2020-05-05 22:36 |
Yes, of course. |
Edward-san (developer) 2020-05-05 22:45 edited on: 2020-05-05 22:47 |
In any case, I'm personally not inclined to change the Scale function to accommodate this specific case. I have a suggestion: does it work if you replace the current multiplication and division in the decal code with the following:
and similar for the y case? Moreover, the Scale call in DBaseDecal::SpreadLeft should be replaced in the same way. |
Only registered users can voice their support. Click here to register, or here to log in. | |
Supporters: | eagle |
Opponents: | No one explicitly opposes this issue yet. |
![]() |
|||
Date Modified | Username | Field | Change |
2019-10-20 07:48 | eagle | New Issue | |
2019-10-20 07:48 | eagle | File Added: CrashReport028.zip | |
2019-12-22 14:34 | Torr Samaho | Note Added: 0021080 | |
2019-12-26 17:09 | eagle | Note Added: 0021081 | |
2020-01-12 20:53 | Torr Samaho | Note Added: 0021082 | |
2020-01-14 07:08 | eagle | File Added: fix1.png | |
2020-01-14 07:09 | eagle | Note Added: 0021083 | |
2020-04-30 20:03 | Torr Samaho | Note Added: 0021295 | |
2020-04-30 20:03 | Torr Samaho | Assigned To | => Torr Samaho |
2020-04-30 20:03 | Torr Samaho | Status | new => needs testing |
2020-05-05 06:45 | eagle | Note Added: 0021309 | |
2020-05-05 06:46 | eagle | File Added: overflow.png | |
2020-05-05 21:22 | Edward-san | Note Added: 0021310 | |
2020-05-05 21:30 | Edward-san | Note Edited: 0021310 | View Revisions |
2020-05-05 21:31 | Edward-san | Note Edited: 0021310 | View Revisions |
2020-05-05 21:58 | eagle | Note Added: 0021312 | |
2020-05-05 21:59 | eagle | Note Added: 0021313 | |
2020-05-05 22:29 | Edward-san | Note Added: 0021315 | |
2020-05-05 22:34 | Edward-san | Note Edited: 0021315 | View Revisions |
2020-05-05 22:36 | eagle | Note Added: 0021316 | |
2020-05-05 22:37 | eagle | File Added: DEMO2.wad | |
2020-05-05 22:45 | Edward-san | Note Added: 0021317 | |
2020-05-05 22:46 | Edward-san | Note Edited: 0021317 | View Revisions |
2020-05-05 22:47 | Edward-san | Note Edited: 0021317 | View Revisions |
2020-05-05 22:47 | Edward-san | Note Edited: 0021317 | View Revisions |
Copyright © 2000 - 2021 MantisBT Team |