Subject: [to-be-updated] framework-for-version-lvb-remove-some-impossible-checks.patch removed from -mm tree To: dan.carpenter@xxxxxxxxxx,jlbec@xxxxxxxxxxxx,mfasheh@xxxxxxxx,rgoldwyn@xxxxxxxx,mm-commits@xxxxxxxxxxxxxxx From: akpm@xxxxxxxxxxxxxxxxxxxx Date: Thu, 21 Nov 2013 16:33:52 -0800 The patch titled Subject: ocfs2: remove some impossible checks has been removed from the -mm tree. Its filename was framework-for-version-lvb-remove-some-impossible-checks.patch This patch was dropped because an updated version will be merged ------------------------------------------------------ From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Subject: ocfs2: remove some impossible checks pv_major and pv_minor are unsigned char type so these checks are just noise. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Cc: Goldwyn Rodrigues <rgoldwyn@xxxxxxxx> Cc: Joel Becker <jlbec@xxxxxxxxxxxx> Cc: Mark Fasheh <mfasheh@xxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/ocfs2/stack_user.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff -puN fs/ocfs2/stack_user.c~framework-for-version-lvb-remove-some-impossible-checks fs/ocfs2/stack_user.c --- a/fs/ocfs2/stack_user.c~framework-for-version-lvb-remove-some-impossible-checks +++ a/fs/ocfs2/stack_user.c @@ -806,11 +806,7 @@ static int lvb_to_version(char *lvb, str &ocfs2_user_plugin.sp_max_proto; memcpy(&pv, lvb, sizeof(struct ocfs2_protocol_version)); - if ((pv.pv_major == LONG_MIN) || (pv.pv_major == LONG_MAX) || - (pv.pv_major > (u8)-1) || (pv.pv_major < 1)) - return -ERANGE; - if ((pv.pv_minor == LONG_MIN) || (pv.pv_minor == LONG_MAX) || - (pv.pv_minor > (u8)-1) || (pv.pv_minor < 0)) + if (pv.pv_major < 1) return -ERANGE; if ((pv.pv_major != max->pv_major) || (pv.pv_minor > max->pv_minor)) _ Patches currently in -mm which might be from dan.carpenter@xxxxxxxxxx are origin.patch drivers-scsi-megaraid-megaraid_mmc-missing-bounds-check-in-mimd_to_kioc.patch drivers-block-paride-pgc-underflow-bug-in-pg_write.patch xfs-underflow-bug-in-xfs_attrlist_by_handle.patch linux-next.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html