)]}'
{"id":"openvpn~1701","triplet_id":"openvpn~master~I2ca8bf90a796f2b757c2fde0ae24468ef3abc3b5","project":"openvpn","branch":"master","attention_set":{},"removed_from_attention_set":{"1000003":{"account":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"last_update":"2026-06-09 10:53:10.000000000","reason":"removed on reply"},"1000030":{"account":{"_account_id":1000030,"name":"MaxF","email":"max@max-fillinger.net","username":"MaxF"},"last_update":"2026-06-10 17:08:38.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I2ca8bf90a796f2b757c2fde0ae24468ef3abc3b5","subject":"Null-terminate tls-crypt client keys when testing","status":"MERGED","created":"2026-05-21 16:43:07.000000000","updated":"2026-06-10 17:08:38.000000000","submitted":"2026-06-10 17:08:38.000000000","submitter":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"1701","meta_rev_id":"5d4aa593e938efdfbec75b421f725b7f6e404e6a","_number":1701,"virtual_id_number":1701,"owner":{"_account_id":1000030,"name":"MaxF","email":"max@max-fillinger.net","username":"MaxF"},"actions":{},"labels":{"Code-Review":{"all":[{"value":0,"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},{"value":0,"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"}],"values":{"-2":"This shall not be submitted","-1":"I would prefer this is not submitted as is"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me, approved"},"description":"","default_value":0}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"}],"CC":[{"_account_id":1000026,"name":"openvpn-devel","email":"openvpn-devel@lists.sourceforge.net","username":"openvpn-devel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-05-21 16:43:07.000000000","updated_by":{"_account_id":1000030,"name":"MaxF","email":"max@max-fillinger.net","username":"MaxF"},"reviewer":{"_account_id":1000026,"name":"openvpn-devel","email":"openvpn-devel@lists.sourceforge.net","username":"openvpn-devel"},"state":"CC"},{"updated":"2026-05-21 16:43:07.000000000","updated_by":{"_account_id":1000030,"name":"MaxF","email":"max@max-fillinger.net","username":"MaxF"},"reviewer":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"state":"REVIEWER"},{"updated":"2026-06-09 10:14:35.000000000","updated_by":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"reviewer":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"state":"REVIEWER"}],"messages":[{"id":"eff23addd3ae4d681eb6eaf161383c9f18f86dea","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":1000030,"name":"MaxF","email":"max@max-fillinger.net","username":"MaxF"},"date":"2026-05-21 16:43:07.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"97cbe45505ae965282ea18dca8277024d01fe492","author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"date":"2026-06-09 10:14:35.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"09a3f08f137362b8cf56133efc24508e8bdf574f","author":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"date":"2026-06-09 10:53:10.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"5d4aa593e938efdfbec75b421f725b7f6e404e6a","tag":"autogenerated:gerrit:merged","author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"date":"2026-06-10 17:08:38.000000000","message":"Change has been successfully pushed.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"b2dcffba0820a1a77472658623b5964e88cbb732","revisions":{"8c28f890128eb7beab953a326abf4f0b82ab24e3":{"kind":"REWORK","_number":1,"created":"2026-05-21 16:43:07.000000000","uploader":{"_account_id":1000030,"name":"MaxF","email":"max@max-fillinger.net","username":"MaxF"},"ref":"refs/changes/01/1701/1","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/01/1701/1","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/1 \u0026\u0026 git checkout -b change-1701 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/01/1701/1","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/1 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"2b8afc6c685f4e451fd0fa5aa37f18147520dfc1","subject":"openvpnserv: always use W variant of RpcStringFree()"}],"author":{"name":"Max Fillinger","email":"maximilian.fillinger@sentyron.com","date":"2026-05-21 15:59:03.000000000","tz":120},"committer":{"name":"Max Fillinger","email":"maximilian.fillinger@sentyron.com","date":"2026-05-21 16:42:02.000000000","tz":120},"subject":"Null-terminate tls-crypt client keys when testing","message":"Null-terminate tls-crypt client keys when testing\n\nAfter generating a tls-crypt-v2 client key, OpenVPN will try to load the\ngenerated key to verify that it was generated correctly. If the client\nkey is not written to disk but printed out on the command line, the PEM\nencoded key is stored in memory and read_pem_key_file is called with\nkey_file_inline \u003d true. However, this key is not a null-terminated\nstring, so we end up calling strlen on a buffer that isn\u0027t\nnull-terminated.\n\nThis commit adds a null-byte at the end of the key.\n\nChange-Id: I2ca8bf90a796f2b757c2fde0ae24468ef3abc3b5\nSigned-off-by: Max Fillinger \u003cmaximilian.fillinger@sentyron.com\u003e\n"},"branch":"refs/heads/master"},"b2dcffba0820a1a77472658623b5964e88cbb732":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":2,"created":"2026-06-10 17:08:38.000000000","uploader":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"ref":"refs/changes/01/1701/2","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/01/1701/2","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/2 \u0026\u0026 git checkout -b change-1701 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/01/1701/2","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/01/1701/2 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"9bad30f6d9884d48db57106ce078bfd7b2a73df9","subject":"mudp: send HMAC reset reply synchronously"}],"author":{"name":"Max Fillinger","email":"maximilian.fillinger@sentyron.com","date":"2026-06-09 10:24:01.000000000","tz":120},"committer":{"name":"Gert Doering","email":"gert@greenie.muc.de","date":"2026-06-10 16:49:14.000000000","tz":120},"subject":"Null-terminate tls-crypt client keys when testing","message":"Null-terminate tls-crypt client keys when testing\n\nAfter generating a tls-crypt-v2 client key, OpenVPN will try to load the\ngenerated key to verify that it was generated correctly. If the client\nkey is not written to disk but printed out on the command line, the PEM\nencoded key is stored in memory and read_pem_key_file is called with\nkey_file_inline \u003d true. However, this key is not a null-terminated\nstring, so we end up calling strlen on a buffer that isn\u0027t\nnull-terminated.\n\nThis commit adds a null-byte at the end of the key.\n\nChange-Id: I2ca8bf90a796f2b757c2fde0ae24468ef3abc3b5\nSigned-off-by: Max Fillinger \u003cmaximilian.fillinger@sentyron.com\u003e\nAcked-by: Gert Doering \u003cgert@greenie.muc.de\u003e\nGerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1701\nMessage-Id: \u003c20260609102407.32590-1-gert@greenie.muc.de\u003e\nURL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg37116.html\nSigned-off-by: Gert Doering \u003cgert@greenie.muc.de\u003e\n"},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
