www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 10622] New: Wrong "this" pointer in methods called via IUnknown derived interfaces

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10622

           Summary: Wrong "this" pointer in methods called via IUnknown
                    derived interfaces
           Product: D
           Version: D2
          Platform: x86_64
        OS/Version: Windows
            Status: NEW
          Severity: major
          Priority: P2
         Component: DMD
        AssignedTo: nobody puremagic.com
        ReportedBy: sludwig outerproduct.org


--- Comment #0 from Sönke Ludwig <sludwig outerproduct.org> 2013-07-12 07:52:57
PDT ---
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

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jul 12 2013
parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10622



--- Comment #1 from Sönke Ludwig <sludwig outerproduct.org> 2013-07-12 08:50:03
PDT ---
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.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jul 12 2013