)]}'
{"id":"LineageOS%2Fandroid_kernel_samsung_exynos5420~286034","triplet_id":"LineageOS%2Fandroid_kernel_samsung_exynos5420~cm-14.1~I84c3ffcc23687c9895a56bba30657f291178c912","project":"LineageOS/android_kernel_samsung_exynos5420","branch":"cm-14.1","topic":"exynos5420-cve","hashtags":[],"change_id":"I84c3ffcc23687c9895a56bba30657f291178c912","subject":"l2tp: fix race between l2tp_session_delete() and l2tp_tunnel_closeall()","status":"MERGED","created":"2020-09-11 08:02:08.000000000","updated":"2020-10-06 10:11:26.000000000","submitted":"2020-10-06 10:11:26.000000000","submitter":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"286035-exynos5420-cve","meta_rev_id":"d211ab65ebb2a2ae857790918ea18fb3950166bd","_number":286034,"virtual_id_number":286034,"owner":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"actions":{},"labels":{"Verified":{"approved":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"all":[{"value":1,"date":"2020-10-06 10:02:19.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-1":"Fails"," 0":"No score","+1":"Verified"},"description":"","default_value":0},"Code-Review":{"approved":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"all":[{"value":2,"date":"2020-10-06 10:02:19.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-2":"Do not submit","-1":"I would prefer that you didn\u0027t submit this"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me, approved"},"description":"","default_value":0},"CI":{"all":[{"value":0,"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-1":"Fail"," 0":"No score","+1":"Pass"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}]},"pending_reviewers":{},"reviewer_updates":[],"messages":[{"id":"e68b04dae2fa8221c6e534ebb07f764f2121543c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2020-09-11 08:02:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"69354e1ec3d1499e187c229ea657a452f722cf4b","author":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2020-10-06 10:02:19.000000000","message":"Patch Set 1: Verified+1 Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"d211ab65ebb2a2ae857790918ea18fb3950166bd","tag":"autogenerated:gerrit:merged","author":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2020-10-06 10:11:26.000000000","message":"Change has been successfully merged by Basil Gello","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"39f82a632f970135273ef113d6e48905f5518c36","revisions":{"39f82a632f970135273ef113d6e48905f5518c36":{"kind":"REWORK","_number":1,"created":"2020-09-11 08:02:08.000000000","uploader":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"ref":"refs/changes/34/286034/1","fetch":{"anonymous http":{"url":"https://github.com/LineageOS/android_kernel_samsung_exynos5420","ref":"refs/changes/34/286034/1","commands":{"Branch":"git fetch https://github.com/LineageOS/android_kernel_samsung_exynos5420 refs/changes/34/286034/1 \u0026\u0026 git checkout -b change-286034 FETCH_HEAD","Checkout":"git fetch https://github.com/LineageOS/android_kernel_samsung_exynos5420 refs/changes/34/286034/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://github.com/LineageOS/android_kernel_samsung_exynos5420 refs/changes/34/286034/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://github.com/LineageOS/android_kernel_samsung_exynos5420 refs/changes/34/286034/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://github.com/LineageOS/android_kernel_samsung_exynos5420 refs/changes/34/286034/1","Reset To":"git fetch https://github.com/LineageOS/android_kernel_samsung_exynos5420 refs/changes/34/286034/1 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"aba85b343defa1f0fb772dfd81554cb3805950c7","subject":"cfg80211: regulatory: reject invalid hints","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_kernel_samsung_exynos5420/commit/aba85b343defa1f0fb772dfd81554cb3805950c7"}]}],"author":{"name":"Guillaume Nault","email":"g.nault@alphalink.fr","date":"2020-04-02 17:32:50.000000000","tz":60},"committer":{"name":"Vasyl Gello","email":"vasek.gello@gmail.com","date":"2020-09-11 08:01:31.000000000","tz":0},"subject":"l2tp: fix race between l2tp_session_delete() and l2tp_tunnel_closeall()","message":"l2tp: fix race between l2tp_session_delete() and l2tp_tunnel_closeall()\n\ncommit b228a94066406b6c456321d69643b0d7ce11cfa6 upstream.\n\nThere are several ways to remove L2TP sessions:\n\n  * deleting a session explicitly using the netlink interface (with\n    L2TP_CMD_SESSION_DELETE),\n  * deleting the session\u0027s parent tunnel (either by closing the\n    tunnel\u0027s file descriptor or using the netlink interface),\n  * closing the PPPOL2TP file descriptor of a PPP pseudo-wire.\n\nIn some cases, when these methods are used concurrently on the same\nsession, the session can be removed twice, leading to use-after-free\nbugs.\n\nThis patch adds a \u0027dead\u0027 flag, used by l2tp_session_delete() and\nl2tp_tunnel_closeall() to prevent them from stepping on each other\u0027s\ntoes.\n\nThe session deletion path used when closing a PPPOL2TP file descriptor\ndoesn\u0027t need to be adapted. It already has to ensure that a session\nremains valid for the lifetime of its PPPOL2TP file descriptor.\nSo it takes an extra reference on the session in the -\u003esession_close()\ncallback (pppol2tp_session_close()), which is eventually dropped\nin the -\u003esk_destruct() callback of the PPPOL2TP socket\n(pppol2tp_session_destruct()).\nStill, __l2tp_session_unhash() and l2tp_session_queue_purge() can be\ncalled twice and even concurrently for a given session, but thanks to\nproper locking and re-initialisation of list fields, this is not an\nissue.\n\nSigned-off-by: Guillaume Nault \u003cg.nault@alphalink.fr\u003e\nSigned-off-by: David S. Miller \u003cdavem@davemloft.net\u003e\nSigned-off-by: Will Deacon \u003cwill@kernel.org\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@linuxfoundation.org\u003e\nSigned-off-by: Lee Jones \u003clee.jones@linaro.org\u003e\n[basilgello: Backport to 3.4]\nSigned-Off-By: Vasyl Gello \u003cvasek.gello@gmail.com\u003e\nChange-Id: I84c3ffcc23687c9895a56bba30657f291178c912\n","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_kernel_samsung_exynos5420/commit/39f82a632f970135273ef113d6e48905f5518c36"}],"resolve_conflicts_web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_kernel_samsung_exynos5420/commit/39f82a632f970135273ef113d6e48905f5518c36"}]},"branch":"refs/heads/cm-14.1"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":17868,"name":"Basil Gello","email":"vasek.gello@gmail.com","username":"gellmar","avatars":[{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/4ebeb3c6587cee55cb7d60233dc9bdbd.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}
