tree fcf500a8622f9de77431d4bbb81d3ebd778ed895
parent 23203b4bb14b3651a490c53c5bf595f6c6b599fd
author Carson Labrado <clabrado@google.com> 1679451960 +0000
committer Carson Labrado <clabrado@google.com> 1679451960 +0000

LogService: Fix potential null pointer dereference

Querying CollectDiagnosticData Action results in a crash if there is
not an active user session.  Fixes the null pointer dereferencing
introduced by https://gerrit.openbmc.org/c/openbmc/bmcweb/+/58688.

Tested:
Queried the Action URI and received a response instead of bmcweb
crashing
curl -X POST -H 'Content-Type: application/json' \
localhost/redfish/v1/Managers/bmc/LogServices/Dump/Actions/LogService.CollectDiagnosticData/ \
-d '{"DiagnosticDataType": "Manager"}'
{
  "@odata.id": "/redfish/v1/TaskService/Tasks/0",
  "@odata.type": "#Task.v1_4_3.Task",
  "Id": "0",
  "TaskState": "Running",
  "TaskStatus": "OK"
}

Signed-off-by: Carson Labrado <clabrado@google.com>
Change-Id: Ic46ece84598613e4bbe51f4188a623ba857c18a0