[caiman-discuss] code review for 5106

[Joseph J VLcek]

Sanjay Nadkarni wrote:
> Please review the updated changes for fix 5106
> 
> http://cr.opensolaris.org/~nadkarni/fix5106-1/webrev/
> 
> -Sanjay
> 
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

usr/src/lib/libti/ti_dm.c

  130         int             ret;

I think ret needs to be initialized.

If the check on line 138 fails because dkinfo.dki_media_type != 
DK_FIXED_DISK
  138         if (dkinfo.dki_media_type == DK_FIXED_DISK) {


I think line 142 will return an unset "ret".

  142         return (ret);