[qpid-dispatch] branch dependabot/npm_and_yarn/console/react/patternfly/react-table-4.19.44 created (now e6e7f3f)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/console/react/patternfly/react-table-4.19.44 in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git. at e6e7f3f Bump @patternfly/react-table from 2.28.50 to 4.19.44 in /console/react No new revisions were added by this update. - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch dependabot/npm_and_yarn/console/react/patternfly/react-core-4.84.3 created (now 87c6e8e)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/console/react/patternfly/react-core-4.84.3 in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git. at 87c6e8e Bump @patternfly/react-core from 3.158.3 to 4.84.3 in /console/react No new revisions were added by this update. - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch dependabot/npm_and_yarn/console/react/patternfly/react-icons-4.7.22 created (now d7db2cd)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/console/react/patternfly/react-icons-4.7.22 in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git. at d7db2cd Bump @patternfly/react-icons from 3.15.17 to 4.7.22 in /console/react No new revisions were added by this update. - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch dependabot/npm_and_yarn/console/react/patternfly/react-charts-6.12.12 created (now 3803846)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/console/react/patternfly/react-charts-6.12.12 in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git. at 3803846 Bump @patternfly/react-charts from 5.3.22 to 6.12.12 in /console/react No new revisions were added by this update. - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch dependabot/npm_and_yarn/console/react/patternfly/patternfly-4.70.2 created (now 094c77f)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/console/react/patternfly/patternfly-4.70.2 in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git. at 094c77f Bump @patternfly/patternfly from 2.71.7 to 4.70.2 in /console/react No new revisions were added by this update. - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch master updated: DISPATCH-1884 - Fixed leaks associated with the deletion of connectors and listeners.
This is an automated email from the ASF dual-hosted git repository. tross pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git The following commit(s) were added to refs/heads/master by this push: new 626a9ef DISPATCH-1884 - Fixed leaks associated with the deletion of connectors and listeners. 626a9ef is described below commit 626a9ef1f3c825f28d652d7f31364e27997bdf24 Author: Ted Ross AuthorDate: Mon Dec 14 13:12:41 2020 -0500 DISPATCH-1884 - Fixed leaks associated with the deletion of connectors and listeners. --- src/adaptors/tcp_adaptor.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/adaptors/tcp_adaptor.c b/src/adaptors/tcp_adaptor.c index 7940705..d813e42 100644 --- a/src/adaptors/tcp_adaptor.c +++ b/src/adaptors/tcp_adaptor.c @@ -718,9 +718,10 @@ static qdr_tcp_connection_t *qdr_tcp_connection_egress(qd_bridge_config_t *confi static void free_bridge_config(qd_bridge_config_t *config) { if (!config) return; +free(config->name); +free(config->address); free(config->host); free(config->port); -free(config->name); free(config->site_id); free(config->host_port); } @@ -732,11 +733,11 @@ static qd_error_t load_bridge_config(qd_dispatch_t *qd, qd_bridge_config_t *conf qd_error_clear(); ZERO(config); -config->name = qd_entity_get_string(entity, "name"); CHECK(); -config->address = qd_entity_get_string(entity, "address"); CHECK(); -config->host = qd_entity_get_string(entity, "host"); CHECK(); -config->port = qd_entity_get_string(entity, "port"); CHECK(); -config->site_id = qd_entity_opt_string(entity, "siteId", 0); CHECK(); +config->name= qd_entity_get_string(entity, "name"); CHECK(); +config->address = qd_entity_get_string(entity, "address"); CHECK(); +config->host= qd_entity_get_string(entity, "host"); CHECK(); +config->port= qd_entity_get_string(entity, "port"); CHECK(); +config->site_id = qd_entity_opt_string(entity, "siteId", 0); CHECK(); int hplen = strlen(config->host) + strlen(config->port) + 2; config->host_port = malloc(hplen); @@ -1249,6 +1250,7 @@ static void qdr_tcp_adaptor_final(void *adaptor_context) qd_tcp_listener_t *tl = DEQ_HEAD(adaptor->listeners); while (tl) { qd_tcp_listener_t *next = DEQ_NEXT(tl); +free_bridge_config(&tl->config); free_qd_tcp_listener_t(tl); tl = next; } @@ -1256,6 +1258,8 @@ static void qdr_tcp_adaptor_final(void *adaptor_context) qd_tcp_connector_t *tr = DEQ_HEAD(adaptor->connectors); while (tr) { qd_tcp_connector_t *next = DEQ_NEXT(tr); +free_bridge_config(&tr->config); +free_qdr_tcp_connection((qdr_tcp_connection_t*) tr->dispatcher); free_qd_tcp_connector_t(tr); tr = next; } - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-proton] branch master updated: PROTON-1516: add tests for empty last frame in a streamed message
This is an automated email from the ASF dual-hosted git repository. cliffjansen pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-proton.git The following commit(s) were added to refs/heads/master by this push: new 8c0e209 PROTON-1516: add tests for empty last frame in a streamed message 8c0e209 is described below commit 8c0e2099cc25c6fb9b53f25335898564a945f9f6 Author: Cliff Jansen AuthorDate: Mon Dec 14 09:59:43 2020 -0800 PROTON-1516: add tests for empty last frame in a streamed message --- c/tests/connection_driver_test.cpp | 42 + python/tests/proton_tests/engine.py | 30 ++ 2 files changed, 72 insertions(+) diff --git a/c/tests/connection_driver_test.cpp b/c/tests/connection_driver_test.cpp index 8a86db5..ff04e81 100644 --- a/c/tests/connection_driver_test.cpp +++ b/c/tests/connection_driver_test.cpp @@ -575,3 +575,45 @@ TEST_CASE("driver_settle_incomplete_receiver") { cond_empty()); CHECK_THAT(*pn_connection_condition(d.server.connection), cond_empty()); } + +/* Empty last frame in streaming message. +*/ +TEST_CASE("driver_empty_last_frame") { + send_client_handler client; + delivery_handler server; + pn_test::driver_pair d(client, server); + + d.run(); + pn_link_t *rcv = server.link; + pn_link_t *snd = client.link; + char data[100] = {0}; /* Dummy data to send. */ + char rbuf[sizeof(data)] = {0}; /* Read buffer for incoming data. */ + pn_link_flow(rcv, 1); + pn_delivery_t *sd = pn_delivery(snd, pn_bytes("1")); /* Prepare to send */ + d.run(); + + /* Send/receive a frame */ + CHECK(sizeof(data) == pn_link_send(snd, data, sizeof(data))); + server.log_clear(); + d.run(); + CHECK_THAT(ETYPES(PN_DELIVERY), Equals(server.log_clear())); + CHECK(sizeof(data) == pn_link_recv(rcv, rbuf, sizeof(data))); + CHECK(pn_delivery_partial(pn_link_current(rcv))); + d.run(); + + /* Advance after all data transfered over wire. */ + CHECK(pn_link_advance(snd)); + server.log_clear(); + d.run(); + CHECK_THAT(ETYPES(PN_DELIVERY), Equals(server.log_clear())); + CHECK(PN_EOS == pn_link_recv(rcv, rbuf, sizeof(data))); + CHECK(!pn_delivery_partial(pn_link_current(rcv))); + + pn_delivery_settle(sd); + sd = NULL; + pn_delivery_settle(pn_link_current(rcv)); + d.run(); + CHECK_THAT(*pn_connection_remote_condition(d.client.connection), + cond_empty()); + CHECK_THAT(*pn_connection_condition(d.server.connection), cond_empty()); +} diff --git a/python/tests/proton_tests/engine.py b/python/tests/proton_tests/engine.py index 1b4c02b..70d10c8 100644 --- a/python/tests/proton_tests/engine.py +++ b/python/tests/proton_tests/engine.py @@ -935,6 +935,36 @@ class TransferTest(Test): self.pump() assert self.rcv.current.aborted + def test_multiframe_last_empty(self): +self.rcv.flow(1) +sd = self.snd.delivery("tag") +msg_p1 = b"this is a test" +n = self.snd.send(msg_p1) +assert n == len(msg_p1) + +self.pump() + +assert len(msg_p1) == self.rcv.current.pending +assert self.rcv.current.partial +msg_p2 = b"this is more." +n = self.snd.send(msg_p2) +assert n == len(msg_p2) + +self.pump() + +msg = msg_p1 + msg_p2 +assert len(msg) == self.rcv.current.pending +assert self.rcv.current.partial +# Advance. Should send empty xfer frame with more flag false. +assert self.snd.advance() + +self.pump() + +assert len(msg) == self.rcv.current.pending +assert not self.rcv.current.partial +binary = self.rcv.recv(self.rcv.current.pending) +assert binary == msg + def test_disposition(self): self.rcv.flow(1) - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
svn commit: r44961 - /dev/qpid/jms/0.56.0-rc1/
Author: robbie Date: Mon Dec 14 17:05:46 2020 New Revision: 44961 Log: add files for qpid-jms 0.56.0 (RC1) Added: dev/qpid/jms/0.56.0-rc1/ dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz (with props) dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.asc (with props) dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.sha512 dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz (with props) dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.asc (with props) dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.sha512 Added: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz == Binary file - no diff available. Propchange: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz -- svn:mime-type = application/gzip Added: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.asc == Binary file - no diff available. Propchange: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.asc -- svn:mime-type = application/pgp-signature Added: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.sha512 == --- dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.sha512 (added) +++ dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-bin.tar.gz.sha512 Mon Dec 14 17:05:46 2020 @@ -0,0 +1 @@ +057daec56d0e822e8f136ca25e36faf2f3924ed66d49979a57dcc8ff99d38f37acf7aee97789028b68dc6564377164b551d8d81bcd5f447c454e83a5315cc6af apache-qpid-jms-0.56.0-bin.tar.gz Added: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz == Binary file - no diff available. Propchange: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz -- svn:mime-type = application/gzip Added: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.asc == Binary file - no diff available. Propchange: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.asc -- svn:mime-type = application/pgp-signature Added: dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.sha512 == --- dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.sha512 (added) +++ dev/qpid/jms/0.56.0-rc1/apache-qpid-jms-0.56.0-src.tar.gz.sha512 Mon Dec 14 17:05:46 2020 @@ -0,0 +1 @@ +eb7ace7f0d9f9858b1f3426c6c8e29f984d5d388e278006f619ee58cea3d1f83eb398b3b8e6e6304667ad3c0c715e7d02ebb622960a678ba4b641d1828dd9c35 apache-qpid-jms-0.56.0-src.tar.gz - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-jms] branch master updated: [maven-release-plugin] prepare for next development iteration
This is an automated email from the ASF dual-hosted git repository. robbie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-jms.git The following commit(s) were added to refs/heads/master by this push: new 927483f [maven-release-plugin] prepare for next development iteration 927483f is described below commit 927483f7ea897a112db27fb032aeadfc58b87a30 Author: Robbie Gemmell AuthorDate: Mon Dec 14 16:18:30 2020 + [maven-release-plugin] prepare for next development iteration --- apache-qpid-jms/pom.xml| 2 +- pom.xml| 4 ++-- qpid-jms-client/pom.xml| 2 +- qpid-jms-discovery/pom.xml | 2 +- qpid-jms-docs/pom.xml | 2 +- qpid-jms-examples/pom.xml | 2 +- qpid-jms-interop-tests/pom.xml | 2 +- qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml | 2 +- 8 files changed, 9 insertions(+), 9 deletions(-) diff --git a/apache-qpid-jms/pom.xml b/apache-qpid-jms/pom.xml index a2c5d60..1613cbc 100644 --- a/apache-qpid-jms/pom.xml +++ b/apache-qpid-jms/pom.xml @@ -18,7 +18,7 @@ org.apache.qpid qpid-jms-parent -0.56.0 +0.57.0-SNAPSHOT 4.0.0 diff --git a/pom.xml b/pom.xml index 97e2844..9fdbd85 100644 --- a/pom.xml +++ b/pom.xml @@ -24,7 +24,7 @@ org.apache.qpid qpid-jms-parent - 0.56.0 + 0.57.0-SNAPSHOT pom QpidJMS 2013 @@ -89,7 +89,7 @@ scm:git:https://gitbox.apache.org/repos/asf/qpid-jms.git scm:git:https://gitbox.apache.org/repos/asf/qpid-jms.git https://gitbox.apache.org/repos/asf?p=qpid-jms.git -0.56.0 +HEAD diff --git a/qpid-jms-client/pom.xml b/qpid-jms-client/pom.xml index 0b8de9e..b7d5198 100644 --- a/qpid-jms-client/pom.xml +++ b/qpid-jms-client/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0 +0.57.0-SNAPSHOT qpid-jms-client diff --git a/qpid-jms-discovery/pom.xml b/qpid-jms-discovery/pom.xml index 42c32ea..321e832 100644 --- a/qpid-jms-discovery/pom.xml +++ b/qpid-jms-discovery/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0 +0.57.0-SNAPSHOT qpid-jms-discovery diff --git a/qpid-jms-docs/pom.xml b/qpid-jms-docs/pom.xml index 3786a32..2bb4996 100644 --- a/qpid-jms-docs/pom.xml +++ b/qpid-jms-docs/pom.xml @@ -18,7 +18,7 @@ org.apache.qpid qpid-jms-parent -0.56.0 +0.57.0-SNAPSHOT 4.0.0 diff --git a/qpid-jms-examples/pom.xml b/qpid-jms-examples/pom.xml index 12a6914..17de51a 100644 --- a/qpid-jms-examples/pom.xml +++ b/qpid-jms-examples/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0 +0.57.0-SNAPSHOT qpid-jms-examples diff --git a/qpid-jms-interop-tests/pom.xml b/qpid-jms-interop-tests/pom.xml index ce30851..11fb8ba 100644 --- a/qpid-jms-interop-tests/pom.xml +++ b/qpid-jms-interop-tests/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0 +0.57.0-SNAPSHOT qpid-jms-interop-tests diff --git a/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml b/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml index 37a2123..1724943 100644 --- a/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml +++ b/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-interop-tests -0.56.0 +0.57.0-SNAPSHOT qpid-jms-activemq-tests - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-jms] annotated tag 0.56.0 created (now 6b2434f)
This is an automated email from the ASF dual-hosted git repository. robbie pushed a change to annotated tag 0.56.0 in repository https://gitbox.apache.org/repos/asf/qpid-jms.git. at 6b2434f (tag) tagging dddaf67a0d97b6a674eba6ad9d238126e2ad8d38 (commit) replaces 0.55.0 by Robbie Gemmell on Mon Dec 14 16:18:19 2020 + - Log - [maven-release-plugin] copy for tag 0.56.0 --- No new revisions were added by this update. - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-jms] branch master updated: [maven-release-plugin] prepare release 0.56.0
This is an automated email from the ASF dual-hosted git repository. robbie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-jms.git The following commit(s) were added to refs/heads/master by this push: new dddaf67 [maven-release-plugin] prepare release 0.56.0 dddaf67 is described below commit dddaf67a0d97b6a674eba6ad9d238126e2ad8d38 Author: Robbie Gemmell AuthorDate: Mon Dec 14 16:17:54 2020 + [maven-release-plugin] prepare release 0.56.0 --- apache-qpid-jms/pom.xml| 2 +- pom.xml| 4 ++-- qpid-jms-client/pom.xml| 2 +- qpid-jms-discovery/pom.xml | 2 +- qpid-jms-docs/pom.xml | 2 +- qpid-jms-examples/pom.xml | 2 +- qpid-jms-interop-tests/pom.xml | 2 +- qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml | 2 +- 8 files changed, 9 insertions(+), 9 deletions(-) diff --git a/apache-qpid-jms/pom.xml b/apache-qpid-jms/pom.xml index 72673a2..a2c5d60 100644 --- a/apache-qpid-jms/pom.xml +++ b/apache-qpid-jms/pom.xml @@ -18,7 +18,7 @@ org.apache.qpid qpid-jms-parent -0.56.0-SNAPSHOT +0.56.0 4.0.0 diff --git a/pom.xml b/pom.xml index 61bc4b5..97e2844 100644 --- a/pom.xml +++ b/pom.xml @@ -24,7 +24,7 @@ org.apache.qpid qpid-jms-parent - 0.56.0-SNAPSHOT + 0.56.0 pom QpidJMS 2013 @@ -89,7 +89,7 @@ scm:git:https://gitbox.apache.org/repos/asf/qpid-jms.git scm:git:https://gitbox.apache.org/repos/asf/qpid-jms.git https://gitbox.apache.org/repos/asf?p=qpid-jms.git -HEAD +0.56.0 diff --git a/qpid-jms-client/pom.xml b/qpid-jms-client/pom.xml index c991eca..0b8de9e 100644 --- a/qpid-jms-client/pom.xml +++ b/qpid-jms-client/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0-SNAPSHOT +0.56.0 qpid-jms-client diff --git a/qpid-jms-discovery/pom.xml b/qpid-jms-discovery/pom.xml index d9b2e81..42c32ea 100644 --- a/qpid-jms-discovery/pom.xml +++ b/qpid-jms-discovery/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0-SNAPSHOT +0.56.0 qpid-jms-discovery diff --git a/qpid-jms-docs/pom.xml b/qpid-jms-docs/pom.xml index adcd2ac..3786a32 100644 --- a/qpid-jms-docs/pom.xml +++ b/qpid-jms-docs/pom.xml @@ -18,7 +18,7 @@ org.apache.qpid qpid-jms-parent -0.56.0-SNAPSHOT +0.56.0 4.0.0 diff --git a/qpid-jms-examples/pom.xml b/qpid-jms-examples/pom.xml index 59470a1..12a6914 100644 --- a/qpid-jms-examples/pom.xml +++ b/qpid-jms-examples/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0-SNAPSHOT +0.56.0 qpid-jms-examples diff --git a/qpid-jms-interop-tests/pom.xml b/qpid-jms-interop-tests/pom.xml index cdb255c..ce30851 100644 --- a/qpid-jms-interop-tests/pom.xml +++ b/qpid-jms-interop-tests/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-parent -0.56.0-SNAPSHOT +0.56.0 qpid-jms-interop-tests diff --git a/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml b/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml index 519eb25..37a2123 100644 --- a/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml +++ b/qpid-jms-interop-tests/qpid-jms-activemq-tests/pom.xml @@ -20,7 +20,7 @@ org.apache.qpid qpid-jms-interop-tests -0.56.0-SNAPSHOT +0.56.0 qpid-jms-activemq-tests - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch master updated: DISPATCH-1879: Fixed use after free issue with logger trying to access freed memory. Also fixed nghttp2 session memory leak. This closes #943.
This is an automated email from the ASF dual-hosted git repository. gmurthy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git The following commit(s) were added to refs/heads/master by this push: new 842e83e DISPATCH-1879: Fixed use after free issue with logger trying to access freed memory. Also fixed nghttp2 session memory leak. This closes #943. 842e83e is described below commit 842e83e88b378d5d321c4e130cd0082e2835928f Author: Ganesh Murthy AuthorDate: Fri Dec 11 14:18:13 2020 -0500 DISPATCH-1879: Fixed use after free issue with logger trying to access freed memory. Also fixed nghttp2 session memory leak. This closes #943. --- src/adaptors/http2/http2_adaptor.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/adaptors/http2/http2_adaptor.c b/src/adaptors/http2/http2_adaptor.c index 161be80..250cf33 100644 --- a/src/adaptors/http2/http2_adaptor.c +++ b/src/adaptors/http2/http2_adaptor.c @@ -365,6 +365,7 @@ void free_qdr_http2_connection(qdr_http2_connection_t* http_conn, bool on_shutdo http_conn->context.context = 0; nghttp2_session_del(http_conn->session_data->session); + free_qdr_http2_session_data_t(http_conn->session_data); sys_mutex_lock(http2_adaptor->lock); DEQ_REMOVE(http2_adaptor->connections, http_conn); @@ -1630,11 +1631,6 @@ uint64_t handle_outgoing_http(qdr_http2_stream_data_t *stream_data) count, stream_data); -for (uint32_t idx = 0; idx < count; idx++) { -free(hdrs[idx].name); -free(hdrs[idx].value); -} - if (stream_id != -1) { stream_data->stream_id = stream_id; } @@ -1651,6 +1647,11 @@ uint64_t handle_outgoing_http(qdr_http2_stream_data_t *stream_data) qd_iterator_free(app_properties_iter); qd_parse_free(app_properties_fld); + +for (uint32_t idx = 0; idx < count; idx++) { +free(hdrs[idx].name); +free(hdrs[idx].value); +} } else { qd_log(http2_adaptor->protocol_log_source, QD_LOG_TRACE, "[C%"PRIu64"][S%"PRId32"] Headers already submitted, Proceeding with the body", conn->conn_id, stream_data->stream_id); @@ -2242,7 +2243,8 @@ static void handle_connection_event(pn_event_t *e, qd_server_t *qd_server, void send_settings_frame(conn); qd_log(log, QD_LOG_INFO, "[C%"PRIu64"] Accepted Ingress ((PN_RAW_CONNECTION_CONNECTED)) from %s", conn->conn_id, conn->remote_address); } else { -nghttp2_session_client_new(&conn->session_data->session, (nghttp2_session_callbacks*)http2_adaptor->callbacks, (void *)conn); +if (!conn->session_data->session) +nghttp2_session_client_new(&conn->session_data->session, (nghttp2_session_callbacks *)http2_adaptor->callbacks, (void *)conn); qd_log(log, QD_LOG_INFO, "[C%"PRIu64"] Connected Egress (PN_RAW_CONNECTION_CONNECTED)", conn->conn_id); conn->connection_established = true; create_stream_dispatcher_link(conn); - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-jms] branch master updated: QPIDJMS-524 Update dependencies to latest
This is an automated email from the ASF dual-hosted git repository. tabish pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-jms.git The following commit(s) were added to refs/heads/master by this push: new 5dbb607 QPIDJMS-524 Update dependencies to latest 5dbb607 is described below commit 5dbb60715d7b05552b2c4ad1ded2d04e28b33495 Author: Timothy Bish AuthorDate: Mon Dec 14 10:15:24 2020 -0500 QPIDJMS-524 Update dependencies to latest slf4j-Log4j2 2.14.0 mockito 3.6.28 --- pom.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pom.xml b/pom.xml index 271a20b..61bc4b5 100644 --- a/pom.xml +++ b/pom.xml @@ -39,7 +39,7 @@ 0.33.8 4.1.55.Final 1.7.30 -2.13.3 +2.14.0 1.0-alpha-2 @@ -51,7 +51,7 @@ 4.13.1 1.0 9.4.22.v20191022 -3.4.0 +3.6.28 2.2 2.9.0 - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch master updated: DISPATCH-1882: TCP test handles empty echo servers dictionary
This is an automated email from the ASF dual-hosted git repository. chug pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git The following commit(s) were added to refs/heads/master by this push: new f632206 DISPATCH-1882: TCP test handles empty echo servers dictionary f632206 is described below commit f6322060d6677415b693ba24b337cfd75cf419c7 Author: Chuck Rolke AuthorDate: Mon Dec 14 09:54:59 2020 -0500 DISPATCH-1882: TCP test handles empty echo servers dictionary --- tests/system_tests_tcp_adaptor.py | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/tests/system_tests_tcp_adaptor.py b/tests/system_tests_tcp_adaptor.py index e73ec9a..236f4c5 100644 --- a/tests/system_tests_tcp_adaptor.py +++ b/tests/system_tests_tcp_adaptor.py @@ -167,6 +167,9 @@ class TcpAdaptor(TestCase): # TCP siteId for listeners and connectors site = "mySite" +# Each router has an echo server to which it connects +echo_servers = {} + @classmethod def setUpClass(cls): """Start a router""" @@ -361,7 +364,6 @@ class TcpAdaptor(TestCase): cls.print_logs_client = True # start echo servers -cls.echo_servers = {} for rtr in cls.router_order: test_name = "TcpAdaptor" server_prefix = "ECHO_SERVER %s ES_%s" % (test_name, rtr) @@ -407,9 +409,10 @@ class TcpAdaptor(TestCase): def tearDownClass(cls): # stop echo servers for rtr in cls.router_order: -server = cls.echo_servers[rtr] -cls.logger.log("TCP_TEST Stopping echo server %s" % rtr) -server.wait() +server = cls.echo_servers.get(rtr) +if not server is None: +cls.logger.log("TCP_TEST Stopping echo server %s" % rtr) +server.wait() super(TcpAdaptor, cls).tearDownClass() # - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org
[qpid-dispatch] branch master updated: DISPATCH-1881 fix build dependencies for http2 protocol adaptor (#945)
This is an automated email from the ASF dual-hosted git repository. jdanek pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git The following commit(s) were added to refs/heads/master by this push: new 723d848 DISPATCH-1881 fix build dependencies for http2 protocol adaptor (#945) 723d848 is described below commit 723d848ee41a479f9c3fd305d48adebf91853517 Author: Jiri Daněk AuthorDate: Mon Dec 14 10:50:24 2020 +0100 DISPATCH-1881 fix build dependencies for http2 protocol adaptor (#945) --- CMakeLists.txt | 4 ++-- cmake/Findlibnghttp2.cmake | 4 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index ef6229f..1f649af 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -57,8 +57,9 @@ endif (NOT APPLE) find_package(PythonInterp REQUIRED) find_package(PythonLibs REQUIRED) find_package(Threads REQUIRED) -find_package(Proton 0.23 REQUIRED COMPONENTS Core Proactor) +find_package(Proton 0.33.0 REQUIRED COMPONENTS Core Proactor) message(STATUS "Found Proton: ${Proton_LIBRARIES} (found version \"${Proton_VERSION}\")" ) +find_package(libnghttp2 1.4.0 REQUIRED) ## ## Optional dependencies @@ -66,7 +67,6 @@ message(STATUS "Found Proton: ${Proton_LIBRARIES} (found version \"${Proton_VERS # Web Sockets find_package(LibWebSockets 2.4.2) -find_package(libnghttp2 1.4.0) CMAKE_DEPENDENT_OPTION(USE_LIBWEBSOCKETS "Use libwebsockets for WebSocket support" ON "LIBWEBSOCKETS_FOUND" OFF) if (LIBWEBSOCKETS_FOUND AND LIBWEBSOCKETS_VERSION_STRING) diff --git a/cmake/Findlibnghttp2.cmake b/cmake/Findlibnghttp2.cmake index fe75579..26fae71 100644 --- a/cmake/Findlibnghttp2.cmake +++ b/cmake/Findlibnghttp2.cmake @@ -19,7 +19,6 @@ find_library(NGHTTP2_LIBRARIES NAMES libnghttp2 nghttp2 - REQUIRED ) find_path(NGHTTP2_INCLUDE_DIRS @@ -31,6 +30,3 @@ find_path(NGHTTP2_INCLUDE_DIRS include(FindPackageHandleStandardArgs) find_package_handle_standard_args( libnghttp2 DEFAULT_MSG NGHTTP2_LIBRARIES NGHTTP2_INCLUDE_DIRS) - - - - To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org For additional commands, e-mail: commits-h...@qpid.apache.org