commit c0f370d1558cf191a113a52f66fa60cdb76e448d
author Kasun Athukorala <kasunath@google.com> Thu Sep 25 07:19:35 2025 +0000
committer Kasun Athukorala <kasunath@google.com> Mon Sep 29 18:37:38 2025 +0000
tree e55cd4fec5fb3674fde36a4ca8e48d5c445308ed
parent 859c28f0b473cbe35ecfe51a899735b401f981ae

Remove file descriptor if fail to get a file pointer

Google-Bug-Id: 423027264
Change-Id: I361bdd50db0ec17871eb1cbb05fdc88be44f7f7e
Signed-off-by: Kasun Athukorala <kasunath@google.com>

libpldmresponder/file_transfer.cpp

diff --git a/libpldmresponder/file_transfer.cpp b/libpldmresponder/file_transfer.cpp
index c2670b9..371a90f 100644
--- a/libpldmresponder/file_transfer.cpp
+++ b/libpldmresponder/file_transfer.cpp
@@ -170,6 +170,7 @@
     int ret = platformHandler->getFilePointer(fileIdentifier, &fp, &fileSize);
     if (ret != 0)
     {
+        removeFileDescriptor(fdOrRet.value());
         error("Requested FileIdentifier: '{FILE_ID}' not found in PDR",
               "FILE_ID", fileIdentifier);
         return CmdHandler::ccOnlyResponse(request,