Toolbar-Auslesen klappt nicht auf einem 64bit-Win7?

[r29d43]

Hallo,

es geht darum, eine Toolbar (die Schnellstartleiste = QuickLaunch) auszulesen, bzw. die Caption-Texte von den Buttons darauf. Bei einem 32bit-Win7 klappte das, beim 64bit-Win7 meines neuen Laptops jetzt aber leider nicht mehr.

Das exakt gleiche Problem hatte ich auch schon beim des Auslesens eines ListView-Controls. Das allerdings konnte ich lösen, mittels eines Tipps (von teebee) aus dem Forum hier (http://www.delphipraxis.net/129414-i...vista-x64.html). Die Lösung hier lag in einem Umbau einer LVItem-Struktur von einer 32Bit-Struktur in eine 64Bit-Struktur.

Ergo versuchte ich diese Lösung 1:1 auf mein ToolBar-Problem zu übertragen, was jetzt aber leider nicht mehr funktionierte. Nach wie vor bekomme ich als Text für die Button-Captions immer wieder nur einen leeren String zurück.

Sieht event. jemand woran das liegen könnte?

Und: Ist meine TBBUTTONINFO64-Struktur auch genauso geworden, wie sie ein 64bit-System erwartet?

Delphi-Quellcode:

function TSmallIconForm.GetTBButtonText64Bit(idCommand: Integer): String;

type
  TBBUTTONINFO64 = packed record
    ewMask: DWORD;
    idCommand: Integer;
    iImage: Integer;
    fsState: Byte;
    fsStyle: Byte;
    cx: Word;
    lParam: DWORD;
//    pszText: PAnsiChar;
                  _align: LongInt; // Zeiger müssen in 64-Bit-Windows an 8-Byte-Alignment beginnen
                  pszText: Int64; // Zeiger haben in 64-Bit-Windows 8 Byte
    cchText: Integer;
  end;
  PTBButtonInfo64 = ^TBButtonInfo64;

var ItemGlob: PTBButtonInfo64;
    InfoStruc: TBButtonInfo64;
    Buffer: Array[0..255] of Char;
    hProc: THandle;
    bw: Cardinal;
begin
  Result := '[Error]';
  if hSchnellstartleiste <> 0 then
  begin
    hProc := OpenProcess(PROCESS_VM_OPERATION or PROCESS_VM_READ or PROCESS_VM_WRITE,false,PID_TaskBar);
    if hProc <> 0 then
    begin
      ItemGlob := VirtualAllocEx(hProc,nil,SizeOf(InfoStruc)+SizeOf(Buffer),
                                 MEM_RESERVE or MEM_COMMIT, PAGE_READWRITE);

      if ItemGlob <> nil then
      try
        with InfoStruc do
        begin
          InfoStruc.cbSize := sizeof(InfoStruc);
          InfoStruc.dwMask := TBIF_TEXT;
          pszText := Cardinal(ItemGlob)+SizeOf(InfoStruc);
          cchText := SizeOf(Buffer);
        end;

        WriteProcessMemory(hProc,ItemGlob,@InfoStruc,SizeOf(InfoStruc),bw);    // parameter zum Data-Transfer: ..., "nach", "von", "länge",...
        sendMessage(hSchnellstartleiste,TB_GETBUTTONINFO,idCommand,integer(ItemGlob));
        ReadProcessMemory(hProc,Pointer(Cardinal(ItemGlob)+SizeOf(InfoStruc)), // parameter zum Data-Transfer: ..., "von", "nach", "länge",...
                          @Buffer[0],SizeOf(Buffer),bw);
        Result := Buffer;
      finally
        VirtualFreeEx(hProc,ItemGlob,SizeOf(InfoStruc)+SizeOf(Buffer),MEM_RELEASE);
        CloseHandle(hProc);
      end;
    end;
  end;
end;

[himitsu]

Mathematik?

Du hast einen 64-Bit-Zeiger und kastest ihn für die Positionsberecchungen nach Integer/Cardinal ... also das kann schonmal nicht gut gehn.

[edit]
hmmmm.

[edit2]

Zitat:

DWORD_PTR lParam;

Pointer?

PS:
- packed
+ {$ALIGN 8}

[r29d43]

Thx erstmal fürs Antworten.


Die TBBUTTONINFO64-Struktur habe ich aus der CommCtrl-Unit, von der dortigen TBBUTTONINFO-Struktur hergeleitet. Und dort steht auch: "lParam: DWORD;"

Das mit dem {$ALIGN 8} hat leider nicht geklappt. Ich glaube auch nicht so richtig dran, dass der Record unpacked sein kann, wenn der beim 32bit-Win schon packed gewesen ist. Es sei denn, beim 64bit-Win wären diese Strukturen jetzt alle unpacked?


Wenn ich übrigens mit SizeOf() die Summe der Längen aller Felder in der Struktur VOR dem pszText-Zeiger ermittle, dann komme ich gemäß:

type
TBBUTTONINFO64 = packed record
cbSize: UINT; = 4Bytes
dwMask: DWORD; = 4Bytes
idCommand: Integer; = 4Bytes
iImage: Integer; = 4Bytes
fsState: Byte; = 1Bytes
fsStyle: Byte; = 1Bytes
cx: Word; = 2Bytes
lParam: DWORD; = 4Bytes
// pszText: PAnsiChar;
_align: longint; // Zeiger müssen in 64-Bit-Windows an 8-Byte-Alignment beginnen
pszText: Int64; // Zeiger haben in 64-Bit-Windows 8 Byte
cchText: Integer;

auf eine gesamte Datenlänge vor dem Ptr von genau 24 Bytes. Also habe ich in einem weiteren Versuch auch mal dieses künstliche "_align: longint" ganz weggelassen. Allerdings auch leider ohne Erfolg.


?

[himitsu]

Das $Align sagt dem Compiler, wie er ausrichten soll, wenn nicht PACKED, womit dann das küntliche _align weg kann.

Was heißt eigentlich "funktioniert nicht" ?


Du könntest/solltest dein InfoStruc eventuell mal Nullen, also FillChar(..., 0) (für alles Ungenutzte/Nichtgesetzte).

Integer-Casts sind sowieso böse und speziell für SendMessage gibt es sogar passede Typen (LPARAM, WPARAM und LRESULT), welche man für den Cast nutzen kann.

Viele Befehle sagen einem, ob sie erfolgreich waren ... man sollte nur mal deren Result und z.B. GetLastError ordentlich auswerten.

Und was sagt der Debugger/Datenexplorer/MemoryView zu den Daten, die am Ende zurückkommen?

[Luckie]

Ist deins ein 32 oder 64 Bit Programm? Und zeig mal etwas Code.

[himitsu]

Zitat von Luckie:

Ist deins ein 32 oder 64 Bit Programm? Und zeig mal etwas Code.

TDE = 32

Ansonsten würde mein erster durchgestrichener Text den Fehler erklären.

[himitsu]

Delphi-Quellcode:

begin
  if hSchnellstartleiste = 0 then
    Exit('[Error]');
  hProc := OpenProcess(PROCESS_VM_OPERATION or PROCESS_VM_READ or PROCESS_VM_WRITE,false,PID_TaskBar);
  if hProc = 0 then
    Exit('[Error:OpenProcess] ' + SysErrorMessage(GetLastError));
  ItemGlob := VirtualAllocEx(hProc, nil, SizeOf(InfoStruc) + SizeOf(Buffer), MEM_RESERVE or MEM_COMMIT or MEM_TOP_DOWN, PAGE_READWRITE);
  if ItemGlob = nil then
    Exit('[Error:VirtualAlloc] ' + SysErrorMessage(GetLastError));
  try
    FillChar(InfoStruc, SizeOf(InfoStruc));  // *1
    with InfoStruc do
    begin
      InfoStruc.cbSize := SizeOf(InfoStruc);
      InfoStruc.dwMask := TBIF_TEXT;
      pszText := IntPtr(ItemGlob) + SizeOf(InfoStruc);
      cchText := SizeOf(Buffer);
    end;
    if not WriteProcessMemory(hProc, ItemGlob, @InfoStruc, SizeOf(InfoStruc), bw) then  // *3
      Exit('[Error:WriteProcessMemory] ' + SysErrorMessage(GetLastError));
    SetLastError(0);  // *4
    R := SendMessage(hSchnellstartleiste, TB_GETBUTTONINFO, idCommand, LPARAM(ItemGlob));
    if (L = 0) or (GetLastError <> 0) then
      Exit('[Error:SendMessage] ' + SysErrorMessage(GetLastError));
    if not ReadProcessMemory(hProc, Pointer(IntPtr(ItemGlob) + SizeOf(InfoStruc)), @Buffer[0], SizeOf(Buffer), bw) then
      Exit('[Error:ReadProcessMemory] ' + SysErrorMessage(GetLastError));
    Result := Buffer;
  finally
    VirtualFreeEx(hProc, ItemGlob, 0, MEM_RELEASE);  // *2 : If the dwFreeType parameter is MEM_RELEASE, dwSize must be 0 (zero)
    CloseHandle(hProc);
  end;
end;

*1) Soll sicherstellen, daß nichtgenutzte Teile 0 sind, damit keine unbeabsichtigten Reaktionen ausgelöst werden ... deine lokale Variable ist aber mit zuzälligem Inhalt versehn.
*2) Dokumentationen sollte man schon noch lesen.
*3) Wie gesagt, Rückgabewerte auswerten und wenn, dann ordentlich, damit man damit auch was anfangen kann.
*4) LastError wird "natürlich" nicht gesetzt, wenn der Aufruf erfolgreich war, aber aus dem Result von SendMessage kann man nicht immer direkt etwas schließen.

Das Exit(...) steht für begin Result := ''; Exit; end; .
IMHO macht es den Code teilweise übersichtlicher.
- Weniger Verschachtelungsebenen, wo man erst viel später im ELSE was finden muß. So sieht man beide Aktionen sofort. (bei Fehler raus und sonst weiter)

[r29d43]

@DeddyH

Grundsätzlich stimme ich dir da natürlich zu. (Wenn allerdings "FindWindow" schon die Taskbar nicht mehr findet, dann ist das System sowieso höchstwahrscheinlich abgestürzt und es wird da wesentlich mehr nicht mehr funktionieren als nur mein Prog.)


@sx2008

Diese Idee hatte ich in der Zwischenzeit tatsächlich auch schon. Allerdings geht es mir in diesem Fall auch einfach etwas ums Prinzip. Dass nämlich ein relativ einfacher Code, der auf einem Win32 bestens läuft, dieses auf einem Win64 jetzt nur wegen eines wahrscheinlich winzigen Fehlers nicht mehr tut... und ich, bevor ich da jetzt also irgendeinen anderen Lösungsweg einschlage, doch lieber erstmal versuche, diesen höchstwahrscheinlich nur "winzigen" Fehler rauszukriegen...


@himitsu

thx für deinen Codehilfe. Leider hat mein Delphi dieses elegante Exit('...') nicht, so dass ich in einem solchen Fall immer so eine relativ globige 4 Zeilen lange begin-end-Struktur brauche.

Ist dein "IntPtr" eigentlich ein normales ^integer? Habe diesbezüglich mal alles etwas ausprobiert, was der Compiler allerdings nie akzeptiert hat. Habe das dann wieder durchs Cardinal ausgetauscht.

Und direkt unter der fraglichen SendMessage-Zeile ist das "if (L=0)..." wohl durch ein "if (R=-1)..." auszutauschen, oder? Denn das L hattest du davor ja nirgendwo initialisiert etc...

Wie auch immer, dass Ergebnis nach dieser SendMessage-Zeile lautet auch mit deinem Code nach wie vor für das R = -1 und für die Übersetzung des GetLastError-Resultates: "[Error: SendMessage] Der Vorgang wurde erfolgreich beendet".

(PS. Hatte den Code mal vor einigen Jahren hier aus dem Forum aufgegabelt und auch ganz gut verstanden ...und deswegen auch nicht nochmal extra in die Dokumentation einer jeden Anweisung reingeguckt.)


@Alle,

imo müsste die eigentliche Fehlerursache immer noch in einer falschen Record-Umwandlung liegen. Weil der doch auch irgendwie das einzigste Teil ist, das sich am ganzen Code wirklich geändert hat! ???

[DeddyH]

Wie gesagt: Rückgaben auswerten und schauen, wo es genau es nicht mehr klappt. Oder möchtest Du lieber so lange raten, bis Du den Fehler scheinbar gefunden hast?

[himitsu]

Schade, ich war mich sicher das Exit wurde schon im D2006 eingeführt.

"Früher" hat man doch Pointer in Integer gecastet, wenn man diesen als Zahl brauchte. Integer(P)
Da nun aber der Integer, in Bezug auf 64 Bit, eingefroren wurde, ist das nicht so schön.
NativeInt könnte man jetzt zwar verwenden, aber da wurde eben auch gleich ein passender Typ für diesen Cast erfunden. IntPtr ist praktisch ein ^NativeInt, bzw. UIntPtr ein ^NativeUInt.
NativeInt/NativeUInt sind die neuen Integer/Cardinal. (Integer war in Win16 ja mal 16-bittig)

Delphi-Quellcode:

{$IF not Declared(IntPtr)}
type
  IntPtr = {$IF Declared(NativeInt)}NativeInt{$ELSE}Integer{$IFEND}
{$IFEND}

Zitat:

Wie auch immer, dass Ergebnis nach dieser SendMessage-Zeile lautet auch mit deinem Code nach wie vor für das R = -1 und für die Übersetzung des GetLastError-Resultates: "[Error: SendMessage] Der Vorgang wurde erfolgreich beendet".

Jupp, sollte ein if (R = -1) bzw. if (R < 0) werden. (SendMessage + TB_GETBUTTONINFO)