change log for rtems (2010-08-24)

[rtems-vc at rtems.org]

 *ralf*:
2010-09-24	Ralf Corsépius <ralf.corsepius at rtems.org>

	* posix/src/aio_read.c: aio_nbytes is always >= 0.

M 1.2588  cpukit/ChangeLog
M    1.4  cpukit/posix/src/aio_read.c

diff -u rtems/cpukit/ChangeLog:1.2587 rtems/cpukit/ChangeLog:1.2588
--- rtems/cpukit/ChangeLog:1.2587	Tue Aug 24 01:46:28 2010
+++ rtems/cpukit/ChangeLog	Tue Aug 24 06:50:34 2010
@@ -1,5 +1,6 @@
 2010-09-24	Ralf Corsépius <ralf.corsepius at rtems.org>
 
+	* posix/src/aio_read.c: aio_nbytes is always >= 0.
 	* posix/src/aio_write.c: aio_nbytes is always >= 0.
 	Fix typo in comment.
 

diff -u rtems/cpukit/posix/src/aio_read.c:1.3 rtems/cpukit/posix/src/aio_read.c:1.4
--- rtems/cpukit/posix/src/aio_read.c:1.3	Mon Aug 16 00:46:09 2010
+++ rtems/cpukit/posix/src/aio_read.c	Tue Aug 24 06:50:35 2010
@@ -51,7 +51,7 @@
   if (aiocbp->aio_reqprio < 0 || aiocbp->aio_reqprio > AIO_PRIO_DELTA_MAX)
     rtems_aio_set_errno_return_minus_one (EINVAL, aiocbp);
   
-  if (aiocbp->aio_offset < 0 || aiocbp->aio_nbytes < 0)
+  if (aiocbp->aio_offset < 0)
     rtems_aio_set_errno_return_minus_one (EINVAL, aiocbp);
 
   req = malloc (sizeof (rtems_aio_request));


 *ralf*:
2010-09-24	Ralf Corsépius <ralf.corsepius at rtems.org>

	* posix/src/aio_cancel.c: Spray pthread_mutex_unlocks.

M 1.2589  cpukit/ChangeLog
M    1.4  cpukit/posix/src/aio_cancel.c

diff -u rtems/cpukit/ChangeLog:1.2588 rtems/cpukit/ChangeLog:1.2589
--- rtems/cpukit/ChangeLog:1.2588	Tue Aug 24 06:50:34 2010
+++ rtems/cpukit/ChangeLog	Tue Aug 24 07:04:43 2010
@@ -1,5 +1,6 @@
 2010-09-24	Ralf Corsépius <ralf.corsepius at rtems.org>
 
+	* posix/src/aio_cancel.c: Spray pthread_mutex_unlocks.
 	* posix/src/aio_read.c: aio_nbytes is always >= 0.
 	* posix/src/aio_write.c: aio_nbytes is always >= 0.
 	Fix typo in comment.
@@ -31,7 +32,6 @@
 
 2010-08-23	Joel Sherrill <joel.sherrill at oarcorp.com>
 
-
 	CID 79/DEADCODE
 	CID 78/DEADCODE
 	CID 77/DEADCODE

diff -u rtems/cpukit/posix/src/aio_cancel.c:1.3 rtems/cpukit/posix/src/aio_cancel.c:1.4
--- rtems/cpukit/posix/src/aio_cancel.c:1.3	Fri Aug 20 04:08:05 2010
+++ rtems/cpukit/posix/src/aio_cancel.c	Tue Aug 24 07:04:43 2010
@@ -45,9 +45,11 @@
 
   if (aiocbp == NULL)
     {
-      if (fcntl (fildes, F_GETFL) < 0)
+      if (fcntl (fildes, F_GETFL) < 0) {
+        pthread_mutex_unlock(&aio_request_queue.mutex);
 	rtems_set_errno_and_return_minus_one (EBADF);
-      
+      }
+
       r_chain = rtems_aio_search_fd (&aio_request_queue.work_req,
 				     fildes,
 				     0);
@@ -58,8 +60,10 @@
 	      r_chain = rtems_aio_search_fd (&aio_request_queue.idle_req,
 					     fildes,
 					     0);
-	      if (r_chain == NULL)
+	      if (r_chain == NULL) {
+	        pthread_mutex_unlock(&aio_request_queue.mutex);
 		return AIO_ALLDONE;
+              }
 
 	      rtems_chain_extract (&r_chain->next_fd);	
 	      rtems_aio_remove_fd (r_chain);
@@ -84,8 +88,7 @@
     }
   else 
     {
-      if (aiocbp->aio_fildes != fildes)
-	{
+      if (aiocbp->aio_fildes != fildes) {
 	  pthread_mutex_unlock (&aio_request_queue.mutex);
 	  rtems_set_errno_and_return_minus_one (EINVAL);
 	}
@@ -120,5 +123,4 @@
     }
   
   return AIO_ALLDONE;
-  
 }



--

Generated by Deluxe Loginfo [http://www.codewiz.org/projects/index.html#loginfo] 2.122 by Bernardo Innocenti <bernie at develer.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/vc/attachments/20100824/9e567ddb/attachment-0001.html>