[PATCH v3 1/2] staging: speakup: reuse native hex2bin()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Andy Shevchenko, le Tue 09 Apr 2013 15:22:15 +0300, a ?crit :
> @@ -349,21 +349,8 @@ char *spk_xlate(char *s)
>  				num = (*p1++)&7;
>  			}
>  			*p++ = num;
> -		} else if (*p1 == 'x' &&
> -				strchr(hx, p1[1]) && strchr(hx, p1[2])) {
...
> +		} else if (*p1 == 'x' && !hex2bin(&num, p1 + 1, 1)) {
> +			p1 += 3;

Checking it again, I realize that this does not check p1[1]!=0 and
p1[2]!=0 any more?; it has to because hex2bin does not perform the
check (and could thus erroneously dereference p1[2]).

Samuel


[Index of Archives]     [Linux for the Blind]     [Fedora Discussioin]     [Linux Kernel]     [Yosemite News]     [Big List of Linux Books]
  Powered by Linux