linux-kernel - Re: Fix test relying in wrong behavior of is

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: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <5B13DF52-26BE-4BA1-8BD7-25863B04A3B6@antoniou-consulting.com>
Date:	Sun, 6 Jan 2013 13:41:56 +0200
From:	Pantelis Antoniou <panto@...oniou-consulting.com>
To:	David Gibson <david@...son.dropbear.id.au>
Cc:	Jon Loeliger <jdl@....com>, Matt Porter <mporter@...com>,
	"devicetree-discuss@...ts.ozlabs.org" 
	<devicetree-discuss@...ts.ozlabs.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Rob Herring <rob.herring@...xeda.com>,
	Russ Dill <Russ.Dill@...com>
Subject: Re: Fix test relying in wrong behavior of is_printable

Hi David

Από το iPhone μου

6 Ιαν 2013, 5:58, ο/η David Gibson <david@...son.dropbear.id.au> έγραψε:

> On Fri, Jan 04, 2013 at 09:16:08PM +0200, Pantelis Antoniou wrote:
>> After fixing the is_printable bug the test suite fails.
>> Fix it with this patch
>> 
>> Signed-off-by: Pantelis Antoniou <panto@...oniou-consulting.com>
> 
> Rather than just removing the test, it would be better to still run it
> using an explicit -t bi to force the byte output.

Perhaps it wasn't exactly clear in the patch. The old test case output is commented out but there is test case output for the new patch. 

So the test is not removed. 


Regards

-- Pantelis
> 
>> ---
>> tests/run_tests.sh | 5 ++---
>> 1 file changed, 2 insertions(+), 3 deletions(-)
>> 
>> diff --git a/tests/run_tests.sh b/tests/run_tests.sh
>> index dd7f217..43279c9 100755
>> --- a/tests/run_tests.sh
>> +++ b/tests/run_tests.sh
>> @@ -498,9 +498,8 @@ fdtget_tests () {
>> 
>>     # run_fdtget_test <expected-result> [<flags>] <file> <node> <property>
>>     run_fdtget_test "MyBoardName" $dtb / model
>> -    run_fdtget_test "77 121 66 111 \
>> -97 114 100 78 97 109 101 0 77 121 66 111 97 114 100 70 97 109 105 \
>> -108 121 78 97 109 101 0" $dtb / compatible
>> +    # run_fdtget_test "77 121 66 111 97 114 100 78 97 109 101 0 77 121 66 111 97 114 100 70 97 109 105 108 121 78 97 109 101 0" $dtb / compatible
>> +    run_fdtget_test "MyBoardName MyBoardFamilyName" $dtb / compatible
>>     run_fdtget_test "MyBoardName MyBoardFamilyName" -t s $dtb / compatible
>>     run_fdtget_test 32768 $dtb /cpus/PowerPC,970@1 d-cache-size
>>     run_fdtget_test 8000 -tx $dtb /cpus/PowerPC,970@1 d-cache-size
> 
> -- 
> David Gibson            | I'll have my music baroque, and my code
> david AT gibson.dropbear.id.au    | minimalist, thank you.  NOT _the_ _other_
>                | _way_ _around_!
> http://www.ozlabs.org/~dgibson
--
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/