Page 1 of 1

Revisione 53623

Posted: Thu Sep 08, 2011 1:34 pm
by Carlo Bramix
Ritengo che sia appena stato fatto il commit di una vaccata.

A parer mio, e' necessario verificare che il path ricevuto sia un volume e, se la diagnosi e' positiva, chiamare semplicemente la funzione GetVolumePathName() per normalizzare il tutto.
Non e' detto che un volume sia solo "C:\", puo' essere qualcosa di visivamente "diverso" che rende semplicemente sbagliato testare se la lunghezza della stringa e' uguale a 2.

http://msdn.microsoft.com/en-us/library ... 85%29.aspx

In ogni caso, dichiarare quella variabile "LocatLen" e chiamare la wcslen() e' pure ridondante in quanto la lunghezza e' gia' nota e pari a "plength - flength": la stringa era stata appena terminata con UNICODE_NULL una riga piu' in alto!
Mah...

A presto,

Carlo Bramini.

Re: Revisione 53623

Posted: Thu Sep 08, 2011 2:20 pm
by gabrielilardi
Sarebbe meglio se lo facessi notare in ros-dev. Ciao.

Re: Revisione 53623

Posted: Fri Sep 09, 2011 12:02 pm
by Carlo Bramix
gabrielilardi wrote:Sarebbe meglio se lo facessi notare in ros-dev. Ciao.
Mah... presto o tardi qualcuno se ne accorgera'...