svn commit: r332070 - in head/sys/geom: . label raid

Allan Jude allanjude at freebsd.org
Fri Apr 6 01:08:09 UTC 2018


On 2018-04-05 09:56, Sean Bruno wrote:
> Author: sbruno
> Date: Thu Apr  5 13:56:40 2018
> New Revision: 332070
> URL: https://svnweb.freebsd.org/changeset/base/332070
> 
> Log:
>   Squash error from geom by sizing ident strings to DISK_IDENT_SIZE.
>   
>   Display attribute in future error strings and differentiate g_handleattr()
>   error messages for ease of debugging in the future.
>   
>   "g_handleattr: md1 bio_length 24 strlen 31 -> EFAULT"
>   
>   Reported by:	swills
>   Reviewed by:	imp cem avg
>   Sponsored by:	Limelight Networks
>   Differential Revision:	https://reviews.freebsd.org/D14962
> 
> Modified:
>   head/sys/geom/geom_subr.c
>   head/sys/geom/label/g_label_disk_ident.c
>   head/sys/geom/raid/md_ddf.c
>   head/sys/geom/raid/md_intel.c
> 
I might be missing something here, but it seems like you didn't actually
change the side of

struct md_s {
...
 char ident[32];
...
}

And I don't see where the destination buffer went from 24 bytes to a
larger value.

-- 
Allan Jude

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20180405/671c67de/attachment.sig>


More information about the svn-src-head mailing list