Bug 10622 – Wrong "this" pointer in methods called via IUnknown derived interfaces

Status
NEW
Severity
major
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
x86_64
OS
Windows
Creation time
2013-07-12T07:52:57Z
Last change time
2024-12-13T18:09:17Z
Assigned to
No Owner
Creator
Sönke Ludwig
Moved to GitHub: dmd#18629 →

Comments

Comment #0 by sludwig — 2013-07-12T07:52:57Z
The following program crashes in the second invocation of AddRef (could be any other method defined by IUnknown or a derived interface) because the "this" pointer points to the IUnknown vtable instead of the object's base, which is assumed by the code. --- import std.c.windows.com; import std.c.windows.windows; class Test : IUnknown { int i = 1; ULONG AddRef() { assert(i == 1); return 0; } ULONG AddRef() { assert(i == 1); return 0; } HRESULT AddRef(IID*, void**) { assert(i == 1); return E_FAIL; } } void main() { auto t = new Test; t.AddRef(); // works auto u = cast(IUnknown)t; u.AddRef(); // crash in _d_invariant } --- Setting this to major severity because it makes defining any COM objects in D impossible on Win64 (and thus many COM APIs are unusable). Tested on DMD 2.063.2
Comment #1 by sludwig — 2013-07-12T08:50:03Z
Actually, as far as I understand, the pointer is correct and the function body is wrong in assuming that it points to the object's base.
Comment #2 by robert.schadek — 2024-12-13T18:09:17Z
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/dmd/issues/18629 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB