)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"change_message_id":"6ba5445fc64fcb4ed1ea834fd95240e617f59c8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fba081fb_e7b53792","updated":"2026-08-13 09:29:00.000000000","message":"In general, I think this is a good change.  It\u0027s incomplete, though, \"git grep\" says... after applying, I still have two occurances\n\n```\nsrc/openvpn/crypto.c:    ASSERT(rand_bytes(output, len));\nsrc/openvpn/ssl_pkt.c:    ASSERT(rand_bytes(key, sizeof(key)));\n```\n\nso I think it makes sense to change those two as well?","commit_id":"1a3b086e44f9eae781d82779dfbcc2952c9e5207"},{"author":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"change_message_id":"6b9f7d6374e35d09dd6fcd31f81573cbb0803506","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1e7f7a20_4495d168","in_reply_to":"fba081fb_e7b53792","updated":"2026-08-13 10:39:30.000000000","message":"The first one is prng_bytes itself:\n\nvoid\nprng_bytes(uint8_t *output, int len)\n{\n    ASSERT(rand_bytes(output, len));\n}\n\nThe second one is something we could do but https://gerrit.openvpn.net/c/openvpn/+/1827 will then conflict since it completely removes the session_id_hmac_init method. I can do that if required but it might be worth the conflict. Or we merge 1827 first, then this one does not matter anyone because it is already gone.","commit_id":"1a3b086e44f9eae781d82779dfbcc2952c9e5207"}]}