Re: [PATCH] ipc: avoid overflow of semop undo (semadj) value

2013-12-03 Thread Davidlohr Bueso
On Tue, 2013-12-03 at 10:44 +0100, Petr Mladek wrote: > When trying to understand semop code, I found a small mistake in the check for > semadj (undo) value overflow. The new undo value is not stored immediately > and next potential checks are done against the old value. > > The failing scenario

Re: [PATCH] ipc: avoid overflow of semop undo (semadj) value

2013-12-03 Thread Davidlohr Bueso
On Tue, 2013-12-03 at 10:44 +0100, Petr Mladek wrote: > When trying to understand semop code, I found a small mistake in the check for > semadj (undo) value overflow. The new undo value is not stored immediately > and next potential checks are done against the old value. > > The failing scenario

[PATCH] ipc: avoid overflow of semop undo (semadj) value

2013-12-03 Thread Petr Mladek
When trying to understand semop code, I found a small mistake in the check for semadj (undo) value overflow. The new undo value is not stored immediately and next potential checks are done against the old value. The failing scenario is not much practical. One semop call has to do more operations

[PATCH] ipc: avoid overflow of semop undo (semadj) value

2013-12-03 Thread Petr Mladek
When trying to understand semop code, I found a small mistake in the check for semadj (undo) value overflow. The new undo value is not stored immediately and next potential checks are done against the old value. The failing scenario is not much practical. One semop call has to do more operations

Re: [PATCH] ipc: avoid overflow of semop undo (semadj) value

2013-12-03 Thread Davidlohr Bueso
On Tue, 2013-12-03 at 10:44 +0100, Petr Mladek wrote: When trying to understand semop code, I found a small mistake in the check for semadj (undo) value overflow. The new undo value is not stored immediately and next potential checks are done against the old value. The failing scenario is

Re: [PATCH] ipc: avoid overflow of semop undo (semadj) value

2013-12-03 Thread Davidlohr Bueso
On Tue, 2013-12-03 at 10:44 +0100, Petr Mladek wrote: When trying to understand semop code, I found a small mistake in the check for semadj (undo) value overflow. The new undo value is not stored immediately and next potential checks are done against the old value. The failing scenario is