Re: [PATCH] udf: limit the maximum number of TD redirections

2015-12-14 Thread Jan Kara
On Mon 14-12-15 11:10:16, Vegard Nossum wrote: > On 12/14/2015 10:52 AM, Jan Kara wrote: > >On Thu 10-12-15 17:13:41, Vegard Nossum wrote: > >>Filesystem fuzzing revealed that we could get stuck in the > >>udf_process_sequence() loop. > >> > >>The maximum limit was chosen arbitrarily but fixes the

Re: [PATCH] udf: limit the maximum number of TD redirections

2015-12-14 Thread Vegard Nossum
On 12/14/2015 10:52 AM, Jan Kara wrote: On Thu 10-12-15 17:13:41, Vegard Nossum wrote: Filesystem fuzzing revealed that we could get stuck in the udf_process_sequence() loop. The maximum limit was chosen arbitrarily but fixes the problem I saw. Process nit: The patch is missing your

Re: [PATCH] udf: limit the maximum number of TD redirections

2015-12-14 Thread Jan Kara
On Thu 10-12-15 17:13:41, Vegard Nossum wrote: > Filesystem fuzzing revealed that we could get stuck in the > udf_process_sequence() loop. > > The maximum limit was chosen arbitrarily but fixes the problem I saw. Process nit: The patch is missing your Signed-off-by. > diff --git fs/udf/super.c

Re: [PATCH] udf: limit the maximum number of TD redirections

2015-12-14 Thread Vegard Nossum
On 12/14/2015 10:52 AM, Jan Kara wrote: On Thu 10-12-15 17:13:41, Vegard Nossum wrote: Filesystem fuzzing revealed that we could get stuck in the udf_process_sequence() loop. The maximum limit was chosen arbitrarily but fixes the problem I saw. Process nit: The patch is missing your

Re: [PATCH] udf: limit the maximum number of TD redirections

2015-12-14 Thread Jan Kara
On Thu 10-12-15 17:13:41, Vegard Nossum wrote: > Filesystem fuzzing revealed that we could get stuck in the > udf_process_sequence() loop. > > The maximum limit was chosen arbitrarily but fixes the problem I saw. Process nit: The patch is missing your Signed-off-by. > diff --git fs/udf/super.c

Re: [PATCH] udf: limit the maximum number of TD redirections

2015-12-14 Thread Jan Kara
On Mon 14-12-15 11:10:16, Vegard Nossum wrote: > On 12/14/2015 10:52 AM, Jan Kara wrote: > >On Thu 10-12-15 17:13:41, Vegard Nossum wrote: > >>Filesystem fuzzing revealed that we could get stuck in the > >>udf_process_sequence() loop. > >> > >>The maximum limit was chosen arbitrarily but fixes the

[PATCH] udf: limit the maximum number of TD redirections

2015-12-10 Thread Vegard Nossum
Filesystem fuzzing revealed that we could get stuck in the udf_process_sequence() loop. The maximum limit was chosen arbitrarily but fixes the problem I saw. --- fs/udf/super.c | 14 ++ 1 file changed, 14 insertions(+) diff --git fs/udf/super.c fs/udf/super.c index 81155b9..fd45537

[PATCH] udf: limit the maximum number of TD redirections

2015-12-10 Thread Vegard Nossum
Filesystem fuzzing revealed that we could get stuck in the udf_process_sequence() loop. The maximum limit was chosen arbitrarily but fixes the problem I saw. --- fs/udf/super.c | 14 ++ 1 file changed, 14 insertions(+) diff --git fs/udf/super.c fs/udf/super.c index 81155b9..fd45537