Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • None
    • None
    • Cache, Core
    • None

    Description

      I had a problem using the update feature. I recevied a SegFault in do_host_db_lookup which was caused by accessing ua_session which was not initialized (see attached patch).

      After fixing that i no longer get an SegFault, but the files that are retrieved by recursion are not placed into the cache. They are requested in every schedule.

      Only the starting file is placed correctly into the cache.

      When retrieving the files with a client, caching works as expected. So i don't think this is a configuration error.

      Attachments

        1. traffic.out
          249 kB
          Thomas Stinner
        2. trafficserver.patch
          0.7 kB
          Thomas Stinner

        Issue Links

          Activity

            jamespeach James Peach added a comment -

            The patch adds a NULL check for HttpSM::ua_session, but ua_session should not be NULL at this point. Maybe there's some condition that is only triggered by the update state machine.

            I took a look at the update code and it doesn't look too promising. I think it will be some work to bring it to life and get it nicely tested. I'm happy to review and land patches.

            jamespeach James Peach added a comment - The patch adds a NULL check for HttpSM::ua_session , but ua_session should not be NULL at this point. Maybe there's some condition that is only triggered by the update state machine. I took a look at the update code and it doesn't look too promising. I think it will be some work to bring it to life and get it nicely tested. I'm happy to review and land patches.
            jamespeach James Peach added a comment -

            Pushing to "sometime". I cancelled the patch because I think the crash here is just a symptom of deeper problems with the update state machine.

            jamespeach James Peach added a comment - Pushing to "sometime". I cancelled the patch because I think the crash here is just a symptom of deeper problems with the update state machine.
            shinrich Susan Hinrichs added a comment -

            Removing prefetch feature gets rid of this issue.

            shinrich Susan Hinrichs added a comment - Removing prefetch feature gets rid of this issue.

            People

              zym Zhao Yongming
              stinnux Thomas Stinner
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Slack

                  Issue deployment