tyler.durden/audk
1
0
Fork 0
mirror of https://github.com/acidanthera/audk.git synced 2025-07-25 14:44:28 +02:00
Code Issues Packages Projects Releases Wiki Activity

Resolved Coverity Issues in Http Dxe

Browse Source 
1.HttpResponseWorker(DEADCODE)
The result of pointer arithmetic "HttpHeaders+AsciiStrLen("HTTP/1.1") + 1"
is never null.
2.HttpsSupport.c (NULL_RETURNS)
NetbufAlloc might return null pointer ,so assigning "NULL" to
"PacketOut" and "DataOut" pointer.

Solution:
1.Removed the NULL Check for "StatusCodeStr"
2.Added Null Check for PacketOut and returned EFI_OUT_OF_RESOURCES
on NULL case.

Signed-off-by: santhosh kumar V <santhoshkumarv@ami.com>
This commit is contained in:
Santhosh Kumar V 2025-01-27 21:02:51 +05:30 committed by mergify[bot]
parent 428cd8a46f
commit df84bb5eda
2 changed files with 35 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
NetworkPkg/HttpDxe/HttpImpl.c
View File

@ -1108,10 +1108,6 @@ HttpResponseWorker (
// Search for Status Code. // Search for Status Code.
// //
StatusCodeStr = HttpHeaders + AsciiStrLen (HTTP_VERSION_STR) + 1; StatusCodeStr = HttpHeaders + AsciiStrLen (HTTP_VERSION_STR) + 1;
if (StatusCodeStr == NULL) {
Status = EFI_NOT_READY;
goto Error;
}
StatusCode = AsciiStrDecimalToUintn (StatusCodeStr); StatusCode = AsciiStrDecimalToUintn (StatusCodeStr);

41
NetworkPkg/HttpDxe/HttpsSupport.c
View File

@ -732,7 +732,6 @@ TlsConfigureSession (
// the caller. The failure is pushed back to TLS DXE driver if the // the caller. The failure is pushed back to TLS DXE driver if the
// HTTP communication actually requires certificate. // HTTP communication actually requires certificate.
// //
Status = EFI_SUCCESS;
} else { } else {
DEBUG ((DEBUG_ERROR, "TLS Certificate Config Error!\n")); DEBUG ((DEBUG_ERROR, "TLS Certificate Config Error!\n"));
return Status; return Status;
@ -1250,7 +1249,13 @@ TlsConnectSession (
// Transmit ClientHello // Transmit ClientHello
// //
PacketOut = NetbufAlloc ((UINT32)BufferOutSize); PacketOut = NetbufAlloc ((UINT32)BufferOutSize);
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (PacketOut == NULL) {
FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES;
}
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (DataOut == NULL) { if (DataOut == NULL) {
FreePool (BufferOut); FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES; return EFI_OUT_OF_RESOURCES;
@ -1336,7 +1341,13 @@ TlsConnectSession (
// Transmit the response packet. // Transmit the response packet.
// //
PacketOut = NetbufAlloc ((UINT32)BufferOutSize); PacketOut = NetbufAlloc ((UINT32)BufferOutSize);
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (PacketOut == NULL) {
FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES;
}
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (DataOut == NULL) { if (DataOut == NULL) {
FreePool (BufferOut); FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES; return EFI_OUT_OF_RESOURCES;
@ -1493,7 +1504,13 @@ TlsCloseSession (
} }
PacketOut = NetbufAlloc ((UINT32)BufferOutSize); PacketOut = NetbufAlloc ((UINT32)BufferOutSize);
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (PacketOut == NULL) {
FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES;
}
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (DataOut == NULL) { if (DataOut == NULL) {
FreePool (BufferOut); FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES; return EFI_OUT_OF_RESOURCES;
@ -1781,7 +1798,13 @@ HttpsReceive (
if (BufferOutSize != 0) { if (BufferOutSize != 0) {
PacketOut = NetbufAlloc ((UINT32)BufferOutSize); PacketOut = NetbufAlloc ((UINT32)BufferOutSize);
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (PacketOut == NULL) {
FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES;
}
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (DataOut == NULL) { if (DataOut == NULL) {
FreePool (BufferOut); FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES; return EFI_OUT_OF_RESOURCES;
@ -1873,7 +1896,13 @@ HttpsReceive (
if (BufferOutSize != 0) { if (BufferOutSize != 0) {
PacketOut = NetbufAlloc ((UINT32)BufferOutSize); PacketOut = NetbufAlloc ((UINT32)BufferOutSize);
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (PacketOut == NULL) {
FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES;
}
DataOut = NetbufAllocSpace (PacketOut, (UINT32)BufferOutSize, NET_BUF_TAIL);
if (DataOut == NULL) { if (DataOut == NULL) {
FreePool (BufferOut); FreePool (BufferOut);
return EFI_OUT_OF_RESOURCES; return EFI_OUT_OF_RESOURCES;