[C] WIN32 Porta seriale / SetDlgItemText

[Gil Mour]

Ho un problema strano

strncpy(buffer,"Opening port",256);
					
SetDlgItemText(hwnd,IDC_LBL_STATE,buffer);

					
serial = CreateFile("COM1",
            GENERIC_READ | GENERIC_WRITE,
            0,
            NULL,
            OPEN_EXISTING,
            0,
            0);

if(serial ==INVALID_HANDLE_VALUE) {
					  
	serial = serial;

}

buffer e serial sono allocati come

char buffer[256];
HANDLE serial;

Ora, se commento la chiamata a SetDlgItemText tutto fila liscio.
Se invece la abilito il programma crasha.

In debug ho visto che con la SetDlgItemText commentata, dopo la CreateFile, serial contiene un indirizzo di memoria 'buono', tipo 0x07a4
Con la SetDlgItemText riabilitata, dopo la CreateFile la variabile serial contiene 0xffffffff.

L'ultima if ha evidentemente solo funzioni di debug

Che succede?

[shodan]

Il problema è qui.

strncpy(buffer,"Opening port",256);

non devi passare la lunghezza del buffer, ma quella della stringa da copiare.

strncpy(buffer,"Opening port",strlen("Opening port")); ricordando che il NULL NON viene copiato.

[Gil Mour]

no è lo stesso

non mi apre l'handle alla seriale se scrivo nel controllo

[menphisx]

Scusa ma serial=serial è un'istruzione inutile.

[Gil Mour]

Grazie della puntualizzazione

ahauhauah

[shodan]

Controlla il valore di ritorno della SetDlgItemText, usa anche GetLastError() e FormatMessage per avere una stringa contenente l'errore.

Altro non si può dire con quel poco codice che hai postato. (Supposto che hwnd sia un handle valido.)

[Gil Mour]

SetDlgItemText restituisce 1
hwnd è un handle valido

il codice che ho postato è praticamente tutto il codice...almeno, tutto il codice che fa qualcosa

[Gil Mour]

Aspetta un attimo azz

Effettivamente. immediatamente dopo aver chiamato setdlgitemtext, GetLastError mi restituisce 5 che equivale alla stringa "Access is denied"

ret = SetDlgItemText(hwnd,IDC_LBL_STATE,buffer);

Ho appena provato a modificare in questo modo

ret = SetDlgItemText(hwnd,MAKEINTRESOURCE(IDC_LBL_STATE),buffer);

La prima volta ha funzionato, poi non più

[Gil Mour]

Ho dimenticato di dire una cosa
Il controllo in questione è una static label

Ora ho provato a sostituirla con un edit box e sembra che non si presenti piu questo problema..