{"id":806169,"url":"http://patchwork.ozlabs.org/api/patches/806169/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20170826225328.7550-2-xypron.glpk@gmx.de/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170826225328.7550-2-xypron.glpk@gmx.de>","list_archive_url":null,"date":"2017-08-26T22:53:20","name":"[U-Boot,11/23] efi_loader: open_info in CloseProtocol","commit_ref":null,"pull_url":null,"state":"superseded","archived":true,"hash":"7f9094d21424fd10bab972cc5e6dba03a83e2089","submitter":{"id":61270,"url":"http://patchwork.ozlabs.org/api/people/61270/?format=json","name":"Heinrich Schuchardt","email":"xypron.glpk@gmx.de"},"delegate":{"id":3400,"url":"http://patchwork.ozlabs.org/api/users/3400/?format=json","username":"agraf","first_name":"Alexander","last_name":"Graf","email":"agraf@suse.de"},"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20170826225328.7550-2-xypron.glpk@gmx.de/mbox/","series":[],"comments":"http://patchwork.ozlabs.org/api/patches/806169/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/806169/checks/","tags":{},"related":[],"headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xfthc6TVCz9s78\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSun, 27 Aug 2017 08:57:20 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 97DF7C221B7; Sat, 26 Aug 2017 22:55:53 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 2BE20C221BA;\n\tSat, 26 Aug 2017 22:55:51 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 91DD5C221E0; Sat, 26 Aug 2017 22:54:22 +0000 (UTC)","from mout.gmx.net (mout.gmx.net [212.227.17.22])\n\tby lists.denx.de (Postfix) with ESMTPS id E3F7CC221D1\n\tfor <u-boot@lists.denx.de>; Sat, 26 Aug 2017 22:54:18 +0000 (UTC)","from localhost.localdomain ([84.118.154.110]) by mail.gmx.com\n\t(mrgmx103 [212.227.17.168]) with ESMTPSA (Nemesis) id\n\t0LeeNW-1dBnR81XTy-00qT3c; Sun, 27 Aug 2017 00:53:36 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable\n\tautolearn_force=no version=3.4.0","From":"Heinrich Schuchardt <xypron.glpk@gmx.de>","To":"Alexander Graf <agraf@suse.de>","Date":"Sun, 27 Aug 2017 00:53:20 +0200","Message-Id":"<20170826225328.7550-2-xypron.glpk@gmx.de>","X-Mailer":"git-send-email 2.14.1","In-Reply-To":"<20170826225328.7550-1-xypron.glpk@gmx.de>","References":"<20170826225110.7381-1-xypron.glpk@gmx.de>\n\t<20170826225328.7550-1-xypron.glpk@gmx.de>","X-Provags-ID":"V03:K0:JL9E6B9SGhc07l+I8iweiuab7q0r1/LkNBrQ2vbly1vnKohgK30\n\tYPsz42S9qE0vGm+wP8h9+XvlgzvA9jNf12kKzQeTN+dLY6Vmn8JIKlYJUwe42YCvX878Fsa\n\t0PB1VM6kOaJBBe5vQ3LAFLd6rhrjR6X2mATydIf0l6EWmI2rfUjvqQJQ/F+4CWgmuTiAUjv\n\t8QUSwosiS9dhjYyQu67SA==","X-UI-Out-Filterresults":"notjunk:1; V01:K0:0R4Fh/8WPx8=:fJ0LO2WoJqKWQOxlxC9plc\n\tx2coI4rGeA58Sfusj95fYyKVwOr5fiQbrdFROY3Ba+HyD6nNPN3ya5SaqCZzn5UV0BucAyKrM\n\tO+ujL0mBiteyf8dgZj4Z/lSK9YxIxHF7f6xPcJSwy8/5LRYzNU8Uh6q9tYhqAuvBBUu/2vwx/\n\tTDal4J3nWq7voKDi48S5m7n9F7lOZWnhTx6vNJx/6g6q1ikF8U7lJ7bPJk9gwMWTl41gqmJte\n\tF05N9sOBfXxPccQimm0kVfX2CRa6YIQkvkxUQlK7/aw3YgODT3E/Aved/8ERsO9S//j27nFbE\n\tl2wpd2IrxGMFYJZaHs1hAePTI5713Y/0z3WQisa4RouNUOEDErRSjDaeKTZApLzkCMnZ0x8sU\n\tMezUXUw9J88T6MZHHj9XSml9w0rbIZDcC+Txh6J4c4Ao90HqyNvSm9rujQNhcaU3fL2xTYnPZ\n\tq/Iqn6XWCxCUJLdfMSBPje/MXKBRzD0GIJQZpSexqg+hL5uy7FqukXmdJs7tdQArs0f88/xLD\n\tGMKTWaer2jFWkEPTcAzuEsJb9JBOz45eDCFtWgXj9rl/0XCB8cuRmmp+r3HOoX9RoYAClWQFO\n\togZtQI3J+gH8FLycdv/5JvbJGiDXGtzYXX8kRxoAuh5aC8jVrvDpDEekyFyQNuXptMjv8F3M/\n\tutNVG+7tNiUkJSIkbnE1YTwC+jE81qOWy7IsDg1hYw802kN1j2Ovs1bArIcgk9eDmGdzwIDeH\n\tFS8T+xrORi6uxbX36176sk4T9ACU1jYGtNuv3Pte1FHajycC3vGzwCU/4VIDwDwHId7DI/By9\n\tiZol6rQfWP7LYzJxBQghrqSFLZSOA==","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>, u-boot@lists.denx.de","Subject":"[U-Boot] [PATCH 11/23] efi_loader: open_info in CloseProtocol","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"},"content":"efi_open_protocol and efi_close_protocol have to keep track of\nopened protocols.\n\nefi_close_protocol has to mark the appropriate entry as\nempty.\n\nSigned-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>\n---\n lib/efi_loader/efi_boottime.c | 33 ++++++++++++++++++++++++++++++++-\n 1 file changed, 32 insertions(+), 1 deletion(-)","diff":"diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c\nindex 294bc1f138..c9aec597a2 100644\n--- a/lib/efi_loader/efi_boottime.c\n+++ b/lib/efi_loader/efi_boottime.c\n@@ -944,9 +944,40 @@ static efi_status_t EFIAPI efi_close_protocol(void *handle,\n \t\t\t\t\t      void *agent_handle,\n \t\t\t\t\t      void *controller_handle)\n {\n+\tstruct efi_handler *handler;\n+\tsize_t i;\n+\tstruct efi_open_protocol_info_entry *open_info;\n+\tefi_status_t r;\n+\n \tEFI_ENTRY(\"%p, %p, %p, %p\", handle, protocol, agent_handle,\n \t\t  controller_handle);\n-\treturn EFI_EXIT(EFI_NOT_FOUND);\n+\n+\tif (!agent_handle) {\n+\t\tr = EFI_INVALID_PARAMETER;\n+\t\tgoto out;\n+\t}\n+\n+\tr = efi_search_protocol(handle, protocol, &handler);\n+\tif (r != EFI_SUCCESS)\n+\t\tgoto out;\n+\n+\tfor (i = 0; i < ARRAY_SIZE(handler->open_info); ++i) {\n+\t\topen_info = &handler->open_info[i];\n+\n+\t\tif (!open_info->open_count)\n+\t\t\tcontinue;\n+\n+\t\tif (open_info->agent_handle == agent_handle &&\n+\t\t    open_info->controller_handle ==\n+\t\t    controller_handle) {\n+\t\t\topen_info->open_count--;\n+\t\t\tr = EFI_SUCCESS;\n+\t\t\tgoto out;\n+\t\t}\n+\t}\n+\tr = EFI_NOT_FOUND;\n+out:\n+\treturn EFI_EXIT(r);\n }\n \n static efi_status_t EFIAPI efi_open_protocol_information(efi_handle_t handle,\n","prefixes":["U-Boot","11/23"]}