lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 15 Sep 2015 23:55:55 +0300
From:	Aaro Koskinen <aaro.koskinen@....fi>
To:	Eric Curtin <ericcurtin17@...il.com>
Cc:	valentina.manea.m@...il.com, shuah.kh@...sung.com,
	linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: tools: usbip: detach: avoid calling strlen() at each iteration

Hi,

On Tue, Sep 15, 2015 at 09:27:20PM +0100, Eric Curtin wrote:
> Instead of calling strlen on every iteration of the for loop, just call it
> once and cache the result in a temporary local variable which will be used
> in the for loop instead.
> 
> Signed-off-by: Eric Curtin <ericcurtin17@...il.com>
> 
> diff --git a/tools/usb/usbip/src/usbip_detach.c b/tools/usb/usbip/src/usbip_detach.c
> index 05c6d15..9db9d21 100644
> --- a/tools/usb/usbip/src/usbip_detach.c
> +++ b/tools/usb/usbip/src/usbip_detach.c
> @@ -47,7 +47,9 @@ static int detach_port(char *port)
>  	uint8_t portnum;
>  	char path[PATH_MAX+1];
>  

The above blank line should be deleted if you declare a new local
variable below...

> -	for (unsigned int i = 0; i < strlen(port); i++)
> +	unsigned int port_len = strlen(port);
> +
> +	for (unsigned int i = 0; i < port_len; i++)

port is read only in this function, so maybe just use "const" and the
compiler should know to do the same without adding a new variable?

A.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists