Module Name: src Committed By: bouyer Date: Wed May 21 21:06:40 UTC 2014
Modified Files: src/sys/fs/cd9660 [netbsd-6-1]: cd9660_node.c Log Message: Pull up following revision(s) (requested by martin in ticket #1062): sys/fs/cd9660/cd9660_node.c: revision 1.31 PR kern/48787: inode calculation from ISO9660 block offset might get truncated to 32bit - force the whole expression to be evaluated as ino_t. Patch from Thomas Schmitt, with minor modifications (and reworded comment). To generate a diff of this commit: cvs rdiff -u -r1.29 -r1.29.22.1 src/sys/fs/cd9660/cd9660_node.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/fs/cd9660/cd9660_node.c diff -u src/sys/fs/cd9660/cd9660_node.c:1.29 src/sys/fs/cd9660/cd9660_node.c:1.29.22.1 --- src/sys/fs/cd9660/cd9660_node.c:1.29 Sun Jun 12 03:35:52 2011 +++ src/sys/fs/cd9660/cd9660_node.c Wed May 21 21:06:40 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: cd9660_node.c,v 1.29 2011/06/12 03:35:52 rmind Exp $ */ +/* $NetBSD: cd9660_node.c,v 1.29.22.1 2014/05/21 21:06:40 bouyer Exp $ */ /*- * Copyright (c) 1982, 1986, 1989, 1994 @@ -37,7 +37,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: cd9660_node.c,v 1.29 2011/06/12 03:35:52 rmind Exp $"); +__KERNEL_RCSID(0, "$NetBSD: cd9660_node.c,v 1.29.22.1 2014/05/21 21:06:40 bouyer Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -438,7 +438,14 @@ isodirino(struct iso_directory_record *i { ino_t ino; - ino = (isonum_733(isodir->extent) + isonum_711(isodir->ext_attr_length)) - << imp->im_bshift; - return (ino); + /* + * Note there is an inverse calculation in + * cd9660_vfsops.c:cd9660_vget_internal(): + * ip->iso_start = ino >> imp->im_bshift; + * and also a calculation of the isodir pointer + * from an inode in cd9660_vnops.c:cd9660_readlink() + */ + ino = ((ino_t)isonum_733(isodir->extent) + + isonum_711(isodir->ext_attr_length)) << imp->im_bshift; + return ino; }