Log4cxx
  1. Log4cxx
  2. LOGCXX-177

SocketImpl::accept uses private APR function: apr_wait_for_io_or_timeout

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None
    • Environment:
      Win32 and standard APR 1.2.7

      Description

      When compiling against standard APR 1.2.7 (1.2.8 should be equivalent on this matter), the symbol apr_wait_for_io_or_timeout is unresolved.

      Searching the apr mailing lists it appears that that function is an internal APR function (unix only btw), which only accidentally found its way to an apr_*.h file. See http://marc.info/?l=apr-dev&m=113309765524699&w=2.

      For projects that already use APR for other things it would be desirable to be able to use a standard APR, instead of one patched for the purposes of log4cxx.

        Activity

        Curt Arnold made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.10.0 [ 10782 ]
        Hide
        Curt Arnold added a comment -

        Removed block that called apr_wait_for_io_or_timeout in rev 588797.

        Could not find a trivial way to reproduce the desired behavior with published APR calls, so any call to SocketImpl::accept will block regardless of any calls to setSoTimeout. Open for better solutions, but not my area of expertise. Would appreciate any comments on the resolution.

        Show
        Curt Arnold added a comment - Removed block that called apr_wait_for_io_or_timeout in rev 588797. Could not find a trivial way to reproduce the desired behavior with published APR calls, so any call to SocketImpl::accept will block regardless of any calls to setSoTimeout. Open for better solutions, but not my area of expertise. Would appreciate any comments on the resolution.
        Diego Santa Cruz created issue -

          People

          • Assignee:
            Curt Arnold
            Reporter:
            Diego Santa Cruz
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development