Re: [PATCH v8 0/2] MR2175: oleaut32: fix VarAbs function for BSTR with positive values
14 Feb
2023
14 Feb
'23
6:18 p.m.
On Tue Feb 14 16:20:32 2023 +0000, Jacek Caban wrote:
This will work, but while reviewing this I noticed that current handling of VT_R4 and VT_R8 is not exactly right, overflow checks don't really make sense. Fixing that allows to simplify VT_BSTR case a bit. Please take a look at https://gitlab.winehq.org/jacek/wine/-/commits/varabs/ and if it looks good to you, update MR with that branch (it also contains BSTR allocation fixup). Thanks. Hopefully I rebased properly!
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/2175#note_24086
1040
Age (days ago)
1040
Last active (days ago)
0 comments
1 participants
participants (1)
-
Jason Millard (@jsm174)