Uploaded image for project: 'Traffic Server'
  1. Traffic Server
  2. TS-1181

TSHttpTxnConfigInt* don't look right with MgmtByte fields in OverridableHttpConfigParams

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.0.4
    • 3.1.4
    • TS API
    • None

    Description

      TSHttpTxnConfigIntSet and Get use _conf_to_memberp to get a pointer to various fields, and then access it as a 32bit int. But many of the fields are now bytes. A fix would be to make _conf_to_memberp return the type properly, and make TSHttpTxnConfigInt* access the field as the proper size.

      I saw valgrind complaining about uninitialized data from the code (which is probably because of padding in the structure, since it is cleared a field at a time), but there are probably much worse effects around reading and writing unrelated fields.

      Attachments

        Activity

          People

            zwoop Leif Hedstrom
            wbardwel William Bardwell
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: