Traffic Server
  1. Traffic Server
  2. TS-915

potential logical problem in net_io, especially for those whose buffer blocks exceed NET_MAX_IOV

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.1
    • Fix Version/s: 3.1.1
    • Component/s: Network
    • Labels:
      None

      Description

      think about this scenario, when read/write socket more than one time in every call read_net_io/write_to_net_io.

        Activity

        Hide
        mohan_zl added a comment -

        Remember i have discussed with weijin, these functions have no problem, we can close this issue, and do nothing. Of course, these functions maybe contain some nonsense lines, like the codes processing "lock.m.m_ptr != s->vio.mutex.m_ptr".

        Show
        mohan_zl added a comment - Remember i have discussed with weijin, these functions have no problem, we can close this issue, and do nothing. Of course, these functions maybe contain some nonsense lines, like the codes processing "lock.m.m_ptr != s->vio.mutex.m_ptr".
        Hide
        Igor Galić added a comment -

        My critique regarding this patch is only formatting, as I have no idea what the problem is, or what the patch solves:

        This patch uses tabs instead of spaces, and at times a mix of both

        Show
        Igor Galić added a comment - My critique regarding this patch is only formatting, as I have no idea what the problem is, or what the patch solves: This patch uses tabs instead of spaces, and at times a mix of both
        Hide
        Leif Hedstrom added a comment -

        Assigning to John for review.

        Show
        Leif Hedstrom added a comment - Assigning to John for review.

          People

          • Assignee:
            John Plevyak
            Reporter:
            weijin
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development