If 'name' then what - we go back to the top of the loop and if there's
more entries then we'll strdup() another name.... Although I have to
wonder if there can be more than non hidden entry in the list... Makes
me wonder about my earlier reviews which didn't think about the loop
exit (that is virSCSIDeviceGetSgName())

It's the only one entry (except the hidden entries), same for
virSCSIDeviceGetSgName. But adding the "break" is better,
to avoid it loops 1 more. Will change when pushing.

Actually adding the break looks a bit strange, and I think we won't care
it tries one more condition checking..
So I pushed this as-is.