Thanks for your response.
This is my test case.
$ ./xfs_io -f mmap -c "pwrite 0 16384" -c "mmap 4096 4096" -c "mread -r"
wrote 16384/16384 bytes at offset 0
16 KiB, 4 ops; 0.0000 sec (625 MiB/sec and 160000.0000 ops/sec)
Segmentation fault
$ cat /etc/SuSE-release
SUSE LINUX 10.0 (X86-64)
VERSION = 10.0
Thu, 30 Nov 2006 09:22:41 +1100 Nathan Scott wrote:
>On Wed, 2006-11-29 at 09:26 +0900, Utako Kusaka wrote:
>> Hi,
>>
>> I found the following issues in xfs_io.
>> mread command:
>> a) Causes a segmentation fault.
>> Because "length"+1 bytes data is copied to buffer in read_mapping(),
>> but buffer size is "length".
>> b) Reads from wrong offset.
>> c) The first byte of dump data is incorrect when length > page size.
>> mwrite command:
>> d) Data placement is incorrect when -r option is specified
>> because of wrong for-loop counter.
>>
>> This patch fixes them.
>>
>
>Looks OK - could you send explicit test cases that demonstrate each
>problem please? (i.e. actual xfs_io invocations). Particularly the
>segfault should be easy to show, something like:
>xfs_io -f -c 'mmap ...' -c 'mread ...' /tmp/foo)
>
>That way they can be added to the regression test suite to ensure these
>things don't spontaneously break themselves in the future.
>
>thanks!
>
>--
>Nathan
|