Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S942408AbcJFJr5 (ORCPT ); Thu, 6 Oct 2016 05:47:57 -0400 Received: from mout.web.de ([212.227.17.12]:61364 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S936596AbcJFJry (ORCPT ); Thu, 6 Oct 2016 05:47:54 -0400 Subject: [PATCH 47/54] md/raid10: Improve another size determination in raid10_start_reshape() To: linux-raid@vger.kernel.org, Christoph Hellwig , Guoqing Jiang , Jens Axboe , Mike Christie , Neil Brown , Shaohua Li , Tomasz Majchrzak References: <566ABCD9.1060404@users.sourceforge.net> <786843ef-4b6f-eb04-7326-2f6f5b408826@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring Message-ID: Date: Thu, 6 Oct 2016 11:47:01 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <786843ef-4b6f-eb04-7326-2f6f5b408826@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:6Mz1dW0wKD9SAAsdsVQYZCl4/+67R+LsjoeZ54OwBbnZ38t6667 NVtMe+PZhf6Qgrsdt72KV22KA2ZBz6WCUWllts68M3Ir4kQqgnc0uzOJUH3DE2ENuRLcNVp dIF/PRYyAjhmo6qyzcyW1HnCn52E5jnWUbO6ULJwSuY//fky5/rY8moknfyhrHGp4bM8/cZ PVbPWqiA5hDgpODMW4yBA== X-UI-Out-Filterresults: notjunk:1;V01:K0:DR1G0vDd2Y0=:ku+OhfjVsKBdAAjfJWGE89 HoQblyeL1MdW8nNikHtYNzk8LM/Dp82os6nLXgZ8FpWepxIJWUm0N2TscirxSX4Ld1ONIrCh5 wObFo5dT7ZWSwwy7TO5rxxcf+A13ztkBItEda8bDtGcFnSl2Mm2IIUIEJdCYLgg3FdQI0F2E6 AWkyU9Q+9i9uQ3hQITJxftTeZYJ5wBgr0a4kHnSbZzKp64kMP1+0Yt03bIygrPbboTkq83Gon Z6BTFvdR4FXE8uhjjxUhzBGJt9p6yDV6glEqUUHn8Mr6ICv1bdxnA4hAbzVcSjm/FogJdnsLn UldJsEsVcQpbcaK71RzWKSF+GOZOYnctILdVX1u8cYRynAJInchQr+fx01pFYkFnwEGKfk+cy Uc+hv7bFyt0wWXMoUX1H7WSA5IXza/4hH5L/W1NomxPaVhBm0kZYJH5vhLgoWQ26dt8Ie43TP Hzl9PI764BH31yQAbSV8fuyDaqL9ZXDgzYxlyZN3XJLz/gFKjhmli0EBs6JFL7Vi+i8hPAyX4 I8MvKYQ12OXZVaBBrlK2lQ0R/rGWHzMi4ZGAOrWF6GU7AjRy4vP63wmKb+0pa3ueAbKLRP+8K BffvjNdCnoApTMhz4k+ZPBlVz1TV/OKe8pVq7htCK/IoYqDg+GQT+R6pgkM/BH2tJ1gOWAmkZ +EAtJDUNoGGsnDbQ/bMqifJau6exjWvELezeXwUq1SZVDVO9hIOoQCsQJzacYRgVY4/2jhzV/ gHcHpCf7kzt1bniRsjXkdb1jhRPzOgbImFfHFCFKo1rZXvyU4O9h+Vzye18SkIwPdkhD8LoD/ 6cAfNMm Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 941 Lines: 27 From: Markus Elfring Date: Wed, 5 Oct 2016 18:48:17 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer. Signed-off-by: Markus Elfring --- drivers/md/raid10.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c index 9b8d11f..62cd159 100644 --- a/drivers/md/raid10.c +++ b/drivers/md/raid10.c @@ -4065,7 +4065,7 @@ static int raid10_start_reshape(struct mddev *mddev) spin_lock_irq(&conf->device_lock); if (conf->mirrors_new) { memcpy(conf->mirrors_new, conf->mirrors, - sizeof(struct raid10_info)*conf->prev.raid_disks); + sizeof(*conf->mirrors_new) * conf->prev.raid_disks); smp_mb(); kfree(conf->mirrors_old); conf->mirrors_old = conf->mirrors; -- 2.10.1