Github user asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/523
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/523#issuecomment-196392204
Created new Jira, https://issues.apache.org/jira/browse/TS-4272. @zwoop
since this is such a simple patch I assume we'll want to backport this? If so,
which ver
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/523#issuecomment-195615709
Yea, once jira is up i'll file one and attach this to it. As for the
ink_zero-- I'll add that to the TODO list of my hostdb cleanup. For now this is
consistent
Github user bgaff commented on the pull request:
https://github.com/apache/trafficserver/pull/523#issuecomment-195615069
I'm not sure this fixes TS4207 but this is a good patch regardless. +1 from
me.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/523#issuecomment-195614866
As per the IRC discussion, this seems unlikely to be our issue, since we
don't enable the hosts files support on our boxes. Also, probably should use
ink_zero() :).
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/523#issuecomment-195615480
Yeah, +1 from me too, but file a new Jira for it since this is likely not
related to TS-4207.
---
If your project is set up for it, you can reply to this email and
GitHub user jacksontj opened a pull request:
https://github.com/apache/trafficserver/pull/523
Potential Fix for TS-4207
std::map is creating an entry, and we are setting a subset of values. Since
we were not clearing the memory that we got, we end up with some
non-initialized struc
