[illumos-Developer] Review for 573 stmfadm doesn't display target and initiator aliases for iscsit

[Deano]

Hi Yuri,
Code looks good to me, has it been tested with more exotic alias names? (CJK
characters for example). Though assuming strdup/free and nvpair handles them
then so should this code.

Bye,
Deano

-----Original Message-----
From: Yuri Pankov [mailto:yuri.pankov at gmail.com] 
Sent: 30 January 2011 19:22
To: developer at lists.illumos.org
Subject: [illumos-Developer] Review for 573 stmfadm doesn't display target
and initiator aliases for iscsit

Hi,

I'm asking for review for 573 stmfadm doesn't display target and
initiator aliases for iscsit:

http://www.illumos.org/issues/573

Webrev:
http://darklight.org.ru/webrev/573-iscsit-aliases/


TIA,
Yuri

_______________________________________________
Developer mailing list
Developer at lists.illumos.org
http://lists.illumos.org/m/listinfo/developer