Re: kern/48799

["Thomas Schmitt" <scdbackup%gmx.net@localhost>]

The following reply was made to PR kern/48799; it has been noted by GNATS.

From: "Thomas Schmitt" <scdbackup%gmx.net@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: kern/48799
Date: Sun, 11 May 2014 12:17:19 +0200

 After finishing my assessment of struct ifid.ifid_start,
 i now understand better the remark of Taylor R Campbell in
   http://mail-index.netbsd.org/tech-kern/2014/05/10/msg017044.html
 "The ifid_start field seems to be unused, incidentally,
  so we needn't even expand the size of cd9660 file handles."
 
 I had the wrong impression that struct fid would be a maximum
 sized wrapper, like a union. But now i see the size inquiry
 protocol of VFS_VPTOFH(,fh_size).
 
 So i will update my patch by a few ifdefs which exclude the
 member .ifid_start from struct ifid by default.
 
 For completeness, i also quote Taylor's other remark:
 "As an aside, it seems from glancing at struct ufid in ufs/ufs/inode.h
  that UFS will have a similar 32-bit overflow problem."