Revisione 53623

Moderators: gabrielilardi, forart, Davy Bartoloni

Post Reply
Carlo Bramix
Posts: 282
Joined: Thu Jan 04, 2007 12:43 am
Location: Italy

Revisione 53623

Post by Carlo Bramix » Thu Sep 08, 2011 1:34 pm

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.

gabrielilardi
Moderator Team
Posts: 873
Joined: Sat Sep 02, 2006 1:30 am
Location: Italy

Re: Revisione 53623

Post by gabrielilardi » Thu Sep 08, 2011 2:20 pm

Sarebbe meglio se lo facessi notare in ros-dev. Ciao.

Carlo Bramix
Posts: 282
Joined: Thu Jan 04, 2007 12:43 am
Location: Italy

Re: Revisione 53623

Post by Carlo Bramix » Fri Sep 09, 2011 12:02 pm

gabrielilardi wrote:Sarebbe meglio se lo facessi notare in ros-dev. Ciao.
Mah... presto o tardi qualcuno se ne accorgera'...

Post Reply

Who is online

Users browsing this forum: Yeti [Bot] and 0 guests