diff options
Diffstat (limited to 'cpukit')
| -rw-r--r-- | cpukit/ChangeLog | 8 | ||||
| -rw-r--r-- | cpukit/libfs/src/pipe/fifo.c | 12 |
2 files changed, 16 insertions, 4 deletions
diff --git a/cpukit/ChangeLog b/cpukit/ChangeLog index f90f41ee95..c8d8a155eb 100644 --- a/cpukit/ChangeLog +++ b/cpukit/ChangeLog @@ -1,3 +1,11 @@ +2010-06-24 Bharath Suri <bharath.s.jois@gmail.com> + + PR 1542/filesystem + PR 1585/filesystem + * libfs/src/pipe/fifo.c: pipe_control_t was not deallocated if + fifo_open() was attempted with (O_WRONLY|O_NONBLOCK). + Mutex was locked too many times on this path and we needed an unlock. + 2010-06-24 Joel Sherrill <joel.sherrilL@OARcorp.com> PR 1587/filesystem diff --git a/cpukit/libfs/src/pipe/fifo.c b/cpukit/libfs/src/pipe/fifo.c index d306c00395..38b90ed36e 100644 --- a/cpukit/libfs/src/pipe/fifo.c +++ b/cpukit/libfs/src/pipe/fifo.c @@ -104,6 +104,8 @@ static int pipe_alloc( if (! pipe->Buffer) goto err_buf; + err = -ENOMEM; + if (rtems_barrier_create( rtems_build_name ('P', 'I', 'r', c), RTEMS_BARRIER_MANUAL_RELEASE, 0, @@ -342,15 +344,17 @@ int fifo_open( break; case LIBIO_FLAGS_WRITE: + pipe->writerCounter ++; + + if (pipe->Writers ++ == 0) + PIPE_WAKEUPREADERS(pipe); + if (pipe->Readers == 0 && LIBIO_NODELAY(iop)) { + PIPE_UNLOCK(pipe); err = -ENXIO; goto out_error; } - pipe->writerCounter ++; - if (pipe->Writers ++ == 0) - PIPE_WAKEUPREADERS(pipe); - if (pipe->Readers == 0) { prevCounter = pipe->readerCounter; err = -EINTR; |