проблемы с GetLogicalDriveStrings - Список дисков

[Alexoid]

Здравствуйте !!!
Чёто немогу сформировать список Дисков, вот код:

    .data
        szDisplayName db "Hello World",0
	szRemoveDrive	db	"  Съёмный диск",			0
    .data?
		szDrive		DD		?
		hH			DD		?
		szDriveOUT	DD		?
    .code
 
 start:
	;Обращение к API-ф-ции WINDOWS MessageBox для вывода сооищения
			CALL GetProcessHeap
		MOV hH,	EAX
		
		PUSH 255
		PUSH HEAP_ZERO_MEMORY
		PUSH hH
			CALL HeapAlloc
		MOV szDriveOUT,	EAX
		
		PUSH OFFSET szDrive
		PUSH 255
			CALL GetLogicalDriveStrings	
		LEA ESI, szDrive
		N4:
			invoke RtlZeroMemory,addr szDriveOUT,255
			LEA EDI, szDriveOUT
			N3:	
				MOV AL,[ESI]
				MOV [EDI],AL
				INC ESI
				INC EDI
			CMP AL,0
			JNZ N3
				invoke MessageBox,0,addr szDriveOUT,0,0
			MOV AL,[ESI]
		CMP AL,0
		JNZ N4
	ret
end start

Если в кратце- список дисков у меня попадает в стоку szDrive, затем мне надо чтоб каждое название диска попало в строку szDriveOUT, чтобы потом с помощью ф-ции GetDriveType Туда ещё добавить тип диска.
И в итоге вывести в COMBOBOX

[B_N]

Сообщение от Alexoid

Если в кратце- список дисков у меня попадает в стоку szDrive, затем мне надо чтоб каждое название диска попало в строку szDriveOUT, чтобы потом с помощью ф-ции GetDriveType Туда ещё добавить тип диска.
И в итоге вывести в COMBOBOX

Тоже вкратце. Никакой список никуда не попадает - посмотрите, для какой строки Вы память выделяете, а в какую хотите записать.

[rpy3uH]

там проблема не только в этом но в самом алгоритме.
надо примерно вот так:

 
  invoke RtlZeroMemory,addr szDriveOUT,255   
  lea edi, szDriveOUT   
N3:  
  mov ax, [ESI]   
  cmp ax, 0
  jz N4
  mov AL,[ESI]   
  MOV [EDI],AL   
  INC ESI   
  INC EDI   
  CMP AL,0   
  JNZ N3   
  invoke MessageBox,0,addr szDriveOUT,0,0     
  jmp N3
N4:

[Alexoid]

Вот немного подправил - Добавил :
PUSH 255
PUSH HEAP_ZERO_MEMORY
PUSH hH
CALL HeapAlloc
MOV szDrive, EAX
да тут я малость не подумал, меня то збило что некоторые API ф-ции выделяют память сами, записывая туда необходимую инфу, и присваивают указатель переменной- что-то вроде этого. по-моему так поступают с некоторыми параметрами advapi ф-ции по работе с реестром или я опять ошибаюсь?
Результат:

    .data
        szDisplayName db "Hello World",0
	szRemoveDrive	db	"  Съёмный диск",			0
    .data?
		szDrive		DD		?
		hH			DD		?
		szDriveOUT	DD		?
    .code
 
 start:
	;Обращение к API-ф-ции WINDOWS MessageBox для вывода сооищения
			CALL GetProcessHeap
		MOV hH,	EAX
		
		PUSH 255
		PUSH HEAP_ZERO_MEMORY
		PUSH hH
			CALL HeapAlloc
		MOV szDrive,	EAX
		
		PUSH 255
		PUSH HEAP_ZERO_MEMORY
		PUSH hH
			CALL HeapAlloc
		MOV szDriveOUT,	EAX
				
		PUSH OFFSET szDrive
		PUSH 254
			CALL GetLogicalDriveStrings	
		LEA ESI, szDrive
		N4:
			invoke RtlZeroMemory,addr szDriveOUT,254
			LEA EDI, szDriveOUT
			N3:	
				MOV AL,[ESI]
				MOV [EDI],AL
				INC ESI
				INC EDI
			CMP AL,0
			JNZ N3
			invoke MessageBox,0,addr szDriveOUT,0,0
			MOV AX,[ESI]
		CMP AX,0
		JNZ N4
	ret
end start

Код Грузина не пошёл может не туда подсталял, но насколко я понимаю идея копирования таже, этот вариант у меня вывел 2 диска из 5.
что скажите?

[B_N]

 PUSH OFFSET szDrive

Вам не смещение, а сам szDrive пушать надо. В нём же адрес строки, а не он сам строка.

[Alexoid]

Вроде ВСЁ попёрло (изменил ещё осталеное - LEA на MOV поменял). Пасибо-ч-к-и %-)

[B_N]

Сообщение от Alexoid

Вроде ВСЁ попёрло (изменил ещё осталеное - LEA на MOV поменял). Пасибо-ч-к-и %-)

Можно ещё, до кучи, убрать теперь копирование строки и просто вызывать MessageBox для фрагментов первой строки...

[Alexoid]

Копирование строки szDrive в szDriveOUT мне необходимо, т.к я хочу ещё с помощью ф-ции GetDriveType к каждому диску добавить описание....
И ещё хотел спросить - после выполнения ф-ции GetLogicalDriveStrings строка szDrive выглядит так: "C:\0D:\0E:\0", где 0 - разделитель, если расчитывать минимальный размер буфера, то получается: в Английском алфавите - 24 буквы, следовательно 24*4=96, так ?
Я незнаю принято ли увас тут просто выкладывать код, вроде как учебных примеров просто думаю, что в целом у меня неплохо получилось(тут и работа со списками и динамическое выдилени памяти и работа со строками - копирование), вот итог:

    .data
	szNoDrive		db	" - Нет устройства",		0
	szRemoveDrive	db	" - Съёмный диск",			0
	szHardDrive		db	" - Жёсткий диск",			0
	szNetDrive		db	" - Сетевой диск",			0
	szCDDrive		db	" - CD-ROM",				0
	szElectDrive	db	" - Электрнный диск",		0
    ;------------------Other
    INDEX1			DD OFFSET szNoDrive
					DD OFFSET szNoDrive
					DD OFFSET szRemoveDrive
					DD OFFSET szHardDrive
					DD OFFSET szNetDrive
					DD OFFSET szCDDrive
					DD OFFSET szElectDrive
					
    .data?
		szDrive		DD		?
		hH			DD		?
		szDriveOUT	DD		?
    .code
 
 start:
			CALL GetProcessHeap
		MOV hH,	EAX
		
		PUSH 255
		PUSH HEAP_ZERO_MEMORY
		PUSH hH
			CALL HeapAlloc
		MOV szDrive,	EAX
		
		PUSH 255
		PUSH HEAP_ZERO_MEMORY
		PUSH hH
			CALL HeapAlloc
		MOV szDriveOUT,	EAX
				
		PUSH szDrive
		PUSH 254
			CALL GetLogicalDriveStrings	
		MOV ESI, szDrive
		N4:
			invoke RtlZeroMemory,szDriveOUT,254
			MOV EDI, szDriveOUT
			N3:	
				MOV AL,[ESI]
				MOV [EDI],AL
				INC ESI
				INC EDI
			CMP AL,0
			JNZ N3
			PUSH ESI			
			PUSH szDriveOUT
				CALL GetDriveType
			MOV EBX,4
			MUL EBX
			MOV ESI,INDEX1[EAX]
			dec edi
			N5:
				MOV AL,[ESI]
				MOV [EDI],AL
				INC ESI
				INC EDI
			CMP AL,0
			JNZ N5
			invoke MessageBox,0,szDriveOUT,0,0
			POP ESI
			MOV AX,[ESI]
		CMP AX,0
		JNZ N4

		PUSH szDrive
		PUSH 1
		PUSH hH
			CALL HeapFree
		PUSH szDriveOUT
		PUSH 1
		PUSH hH
			CALL HeapFree
	ret
end start

[B_N]

Сообщение от Alexoid

И ещё хотел спросить - после выполнения ф-ции GetLogicalDriveStrings строка szDrive выглядит так: "C:\0D:\0E:\0", где 0 - разделитель, если расчитывать минимальный размер буфера, то получается: в Английском алфавите - 24 буквы, следовательно 24*4=96, так ?

И ещё один '\0', завершающий всю эту компанию - "c:\<null>d:\<null>m:\<null><null>" . Только в этих расчетах нет никакой нужды - функция сама говорит, какой ей нужен буфер, если данного ей недостаточно - проще говоря, если вызвать её с нулями. Ну а после "нормального" вызова не грех убедиться, что она вернула значение, не большее, чем длина буфера.
http://msdn2.microsoft.com/en-us/lib...75(VS.85).aspx

[Alexoid]

Я правелно Вас понял, если её вызвать вот так:
PUSH 0
PUSH 0
CALL GetLogicalDriveStrings
то eax будет содержать число-размер буфера щас попробую.... да фантастика, ещё раз убеждаюсь, что WINAPI - Это СИЛА, с 5-ю дисками он выдал 21, а с 6 -25, что и выходит: 25-1=24(убрал последний ноль) 24/6=4 - каждый диск по 4 байта. Ну это конечно уже решать тому кто прогу делает либо 2 раза вызывать GetLogicalDriveStrings и терять драгоциенные такты процессора, но выделять точное кол-во памяти; либо выделить максимум и прибавить еще 5-6 байтов для уверенности, тем более если размер буфера не такой уж и большой.
До самой истины дошли, тут на этом посте можно Диссертацию накатать покрайней мере основа положена