You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by jp...@apache.org on 2016/10/06 04:09:14 UTC
[trafficserver] branch master updated (489faa3 -> 2cef677)
This is an automated email from the ASF dual-hosted git repository.
jpeach pushed a change to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git.
from 489faa3 Make local_event_callbacks variable static.
new 3772683 TS-4925: Fix static analysis check.
new 2cef677 Fix default Darwin build options.
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.
Summary of changes:
ci/regression | 1 +
mgmt/LocalManager.cc | 4 +++-
2 files changed, 4 insertions(+), 1 deletion(-)
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].
[trafficserver] 02/02: Fix default Darwin build options.
Posted by jp...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
jpeach pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
commit 2cef6779679fde16565dbdf6ce8ab3586fa53886
Author: James Peach <jp...@apache.org>
AuthorDate: Wed Oct 5 21:09:05 2016 -0700
Fix default Darwin build options.
---
ci/regression | 1 +
1 file changed, 1 insertion(+)
diff --git a/ci/regression b/ci/regression
index 51fc1de..14a6b8b 100755
--- a/ci/regression
+++ b/ci/regression
@@ -69,6 +69,7 @@ prog() {
extras() {
case $(uname -v) in
omnios*) echo --with-tcl=/opt/omni/lib/amd64 ;;
+ Darwin*) echo --with-openssl=$(brew --prefix openssl) ;;
esac
}
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.
[trafficserver] 01/02: TS-4925: Fix static analysis check.
Posted by jp...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
jpeach pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
commit 3772683de978d85f545f3bb9afa3fca6e793c0b6
Author: James Peach <jp...@apache.org>
AuthorDate: Wed Oct 5 21:08:48 2016 -0700
TS-4925: Fix static analysis check.
---
mgmt/LocalManager.cc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/mgmt/LocalManager.cc b/mgmt/LocalManager.cc
index f90b342..8e91071 100644
--- a/mgmt/LocalManager.cc
+++ b/mgmt/LocalManager.cc
@@ -43,12 +43,14 @@ void
LocalManager::mgmtCleanup()
{
close_socket(process_server_sockfd);
+ process_server_sockfd = ts::NO_FD;
// fix me for librecords
if (virt_map) {
virt_map->rl_downAddrs(); // We are bailing done need to worry about table
}
+
closelog();
return;
}
@@ -441,7 +443,7 @@ LocalManager::pollMgmtProcessServer()
}
#endif
- if (FD_ISSET(process_server_sockfd, &fdlist)) { /* New connection */
+ if (process_server_sockfd != ts::NO_FD && FD_ISSET(process_server_sockfd, &fdlist)) { /* New connection */
struct sockaddr_in clientAddr;
socklen_t clientLen = sizeof(clientAddr);
int new_sockfd = mgmt_accept(process_server_sockfd, (struct sockaddr *)&clientAddr, &clientLen);
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.