Fix bmcweb core-dump caused by Split up authenticate
Ed changed the code from req.emplace(parser->release()) to
req.reset() in line 728 of the http/http_connection.hpp file in this
patch https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/47122.
So req cannot be used in doReadHeaders(). These codes are not necessary,
so I choose to delete them.
Tested:
1. Keep these codes and set bmcweb-logging=enabled, then log in to
bmcweb using webui. Bmcweb will core-dump.
2. Delete these codes, bmcweb works normally.
Signed-off-by: zhanghaicheng <zhanghch05@inspur.com>
Change-Id: I1875a3fe4fa1d03656631435508b3876d8a42e54
diff --git a/http/http_connection.hpp b/http/http_connection.hpp
index fb5eac7..2e9ff38 100644
--- a/http/http_connection.hpp
+++ b/http/http_connection.hpp
@@ -584,17 +584,6 @@
{
startDeadline(loggedInAttempts);
BMCWEB_LOG_DEBUG << "Starting slow deadline";
-
- req->urlParams = req->urlView.query_params();
-
-#ifdef BMCWEB_ENABLE_DEBUG
- std::string paramList = "";
- for (const auto param : req->urlParams)
- {
- paramList += param->key() + " " + param->value() + " ";
- }
- BMCWEB_LOG_DEBUG << "QueryParams: " << paramList;
-#endif
}
else
{