From patchwork Tue Apr 30 04:59:19 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Jon ERNST X-Patchwork-Id: 240550 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 748132C00AD for ; Tue, 30 Apr 2013 14:59:28 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751406Ab3D3E7V (ORCPT ); Tue, 30 Apr 2013 00:59:21 -0400 Received: from mout.gmx.net ([74.208.4.201]:55877 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751254Ab3D3E7V (ORCPT ); Tue, 30 Apr 2013 00:59:21 -0400 Received: from mailout-us.gmx.com ([172.19.198.46]) by mrigmx.server.lan (mrigmxus001) with ESMTP (Nemesis) id 0MhAVJ-1UAiSH1cJd-00MOhT for ; Tue, 30 Apr 2013 06:59:20 +0200 Received: (qmail 11251 invoked by uid 0); 30 Apr 2013 04:59:20 -0000 Received: from 72.219.205.124 by rms-us002 with HTTP Date: Tue, 30 Apr 2013 00:59:19 -0400 From: "Jon ERNST" Message-ID: <20130430045920.296280@gmx.com> MIME-Version: 1.0 Subject: [ext4] extent tree es_len can be zero? To: linux-ext4@vger.kernel.org X-Flags: 0001 X-Mailer: GMX.com Web Mailer x-registered: 0 X-GMX-UID: /yoNcdgS3zOlNclmlXwhGYp+IGRvb0Ca Sender: linux-ext4-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org Hi ,list, extent tree es_len can be zero? if we call ext4_es_end when es_len is zero, it will have problem. the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/fs/ext4/extents_status.c b/fs/ext4/extents_status.c index 95796a1..49a8da2 100644 --- a/fs/ext4/extents_status.c +++ b/fs/ext4/extents_status.c @@ -108,7 +108,7 @@  *     tree are ordered by logical block no.  *  *   --        operations on a extent status tree - *     There are three important operations on a delayed extent tree: find + *     There are three important operations on a delayed extent tree: finding  *     next extent, adding a extent(a range of blocks) and removing a extent.  *  *   --        race on a extent status tree @@ -195,7 +195,7 @@ static void ext4_es_print_tree(struct inode *inode) static inline ext4_lblk_t ext4_es_end(struct extent_status *es) { -       BUG_ON(es->es_lblk + es->es_len < es->es_lblk); +       BUG_ON(es->es_lblk + es->es_len <= es->es_lblk);        return es->es_lblk + es->es_len - 1; } -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in