Commit 5871d0c6 authored by Dan Carpenter's avatar Dan Carpenter Committed by David S. Miller
Browse files

ionic: return -EFAULT if copy_to_user() fails 



The copy_to_user() function returns the number of bytes that it wasn't
able to copy.  We want to return -EFAULT to the user.

Fixes: fee6efce ("ionic: add hw timestamp support files")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarShannon Nelson <snelson@pensando.io>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 23ba5117

drivers/net/ethernet/pensando/ionic/ionic_phc.c

+3 −1
Original line number Diff line number Diff line
@@ -225,7 +225,9 @@ int ionic_lif_hwstamp_get(struct ionic_lif *lif, struct ifreq *ifr) 
	memcpy(&config, &lif->phc->ts_config, sizeof(config));

	mutex_unlock(&lif->phc->config_lock);



	return copy_to_user(ifr->ifr_data, &config, sizeof(config));

	if (copy_to_user(ifr->ifr_data, &config, sizeof(config)))

		return -EFAULT;

	return 0;

}



static u64 ionic_hwstamp_read(struct ionic *ionic,