Bug 9655 - source3/lib/system.c has wrong test for aio_suspend (uses HAVE_AIO_FSYNC)
Summary: source3/lib/system.c has wrong test for aio_suspend (uses HAVE_AIO_FSYNC)
Status: NEW
Alias: None
Product: Samba 3.6
Classification: Unclassified
Component: File services (show other bugs)
Version: unspecified
Hardware: All All
: P5 normal
Target Milestone: ---
Assignee: Volker Lendecke
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-12 19:07 UTC by Richard Sharpe
Modified: 2013-02-12 19:07 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Sharpe 2013-02-12 19:07:14 UTC
In looking at the FreeBSD ports collection, I see that Timur has already fixed a minor problem.

Here is his patch:

Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>

--- ./source3/lib/system.c.orig 2011-08-09 13:17:47.000000000 +0200
+++ ./source3/lib/system.c      2011-10-13 23:38:02.000000000 +0200
@@ -2586,14 +2586,14 @@
 }

 /*******************************************************************
- An aio_fsync wrapper that will deal with 64-bit sizes.
+ An aio_suspend wrapper that will deal with 64-bit sizes.
 ********************************************************************/

 int sys_aio_suspend(const SMB_STRUCT_AIOCB * const cblist[], int n, const struct timespec *timeout)
 {
 #if defined(HAVE_EXPLICIT_LARGEFILE_SUPPORT) && defined(HAVE_AIOCB64) && defined(HAVE_AIO_SUSPEND64)
         return aio_suspend64(cblist, n, timeout);
-#elif defined(HAVE_AIO_FSYNC)
+#elif defined(HAVE_AIO_SUSPEND)
         return aio_suspend(cblist, n, timeout);
 #else
        errno = ENOSYS;