VirtualBox

Ignore:
Timestamp:
Apr 14, 2023 3:17:44 PM (22 months ago)
Author:
vboxsync
Message:

Devices/EFI/FirmwareNew: Update to edk2-stable202302 and make it build, bugref:4643

Location:
trunk/src/VBox/Devices/EFI/FirmwareNew
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Devices/EFI/FirmwareNew

  • trunk/src/VBox/Devices/EFI/FirmwareNew/NetworkPkg/Mtftp6Dxe/Mtftp6Option.c

    r85718 r99404  
    1010#include "Mtftp6Impl.h"
    1111
    12 CHAR8 *mMtftp6SupportedOptions[MTFTP6_SUPPORTED_OPTIONS_NUM] = {
     12CHAR8  *mMtftp6SupportedOptions[MTFTP6_SUPPORTED_OPTIONS_NUM] = {
    1313  "blksize",
    1414  "windowsize",
    … …  
    1818};
    1919
    20 
    2120/**
    2221  Parse the NULL terminated ASCII string of multicast option.
    … …  
    3332EFI_STATUS
    3433Mtftp6ParseMcastOption (
    35   IN UINT8                  *Str,
    36   IN MTFTP6_EXT_OPTION_INFO *ExtInfo
     34  IN UINT8                   *Str,
     35  IN MTFTP6_EXT_OPTION_INFO  *ExtInfo
    3736  )
    3837{
    39   EFI_STATUS                Status;
    40   UINT32                    Num;
    41   CHAR8                     *Ip6Str;
    42   CHAR8                     *TempStr;
     38  EFI_STATUS  Status;
     39  UINT32      Num;
     40  CHAR8       *Ip6Str;
     41  CHAR8       *TempStr;
    4342
    4443  //
    … …  
    4746  //
    4847  if (*Str == ',') {
    49 
    5048    ZeroMem (&ExtInfo->McastIp, sizeof (EFI_IPv6_ADDRESS));
    5149  } else {
    52 
    53     Ip6Str = (CHAR8 *) AllocateCopyPool (AsciiStrSize ((CHAR8 *) Str), Str);
     50    Ip6Str = (CHAR8 *)AllocateCopyPool (AsciiStrSize ((CHAR8 *)Str), Str);
    5451    if (Ip6Str == NULL) {
    5552      return EFI_OUT_OF_RESOURCES;
    … …  
    8986  //
    9087  if (*Str == ',') {
    91 
    9288    ExtInfo->McastPort = 0;
    9389  } else {
    94 
    95     Num = (UINT32) AsciiStrDecimalToUintn ((CHAR8 *) Str);
     90    Num = (UINT32)AsciiStrDecimalToUintn ((CHAR8 *)Str);
    9691
    9792    if (Num > 65535) {
    … …  
    9994    }
    10095
    101     ExtInfo->McastPort = (UINT16) Num;
     96    ExtInfo->McastPort = (UINT16)Num;
    10297
    10398    while (NET_IS_DIGIT (*Str)) {
    … …  
    115110  // Check the master/slave setting, 1 for master, 0 for slave.
    116111  //
    117   Num = (UINT32) AsciiStrDecimalToUintn ((CHAR8 *) Str);
    118 
    119   if (Num != 0 && Num != 1) {
    120     return EFI_INVALID_PARAMETER;
    121   }
    122 
    123   ExtInfo->IsMaster = (BOOLEAN) (Num == 1);
     112  Num = (UINT32)AsciiStrDecimalToUintn ((CHAR8 *)Str);
     113
     114  if ((Num != 0) && (Num != 1)) {
     115    return EFI_INVALID_PARAMETER;
     116  }
     117
     118  ExtInfo->IsMaster = (BOOLEAN)(Num == 1);
    124119
    125120  while (NET_IS_DIGIT (*Str)) {
    … …  
    133128  return EFI_SUCCESS;
    134129}
    135 
    136130
    137131/**
    … …  
    152146EFI_STATUS
    153147Mtftp6ParseExtensionOption (
    154   IN EFI_MTFTP6_OPTION        *Options,
    155   IN UINT32                   Count,
    156   IN BOOLEAN                  IsRequest,
    157   IN UINT16                   Operation,
    158   IN MTFTP6_EXT_OPTION_INFO   *ExtInfo
     148  IN EFI_MTFTP6_OPTION       *Options,
     149  IN UINT32                  Count,
     150  IN BOOLEAN                 IsRequest,
     151  IN UINT16                  Operation,
     152  IN MTFTP6_EXT_OPTION_INFO  *ExtInfo
    159153  )
    160154{
    161   EFI_STATUS                  Status;
    162   EFI_MTFTP6_OPTION           *Opt;
    163   UINT32                      Index;
    164   UINT32                      Value;
     155  EFI_STATUS         Status;
     156  EFI_MTFTP6_OPTION  *Opt;
     157  UINT32             Index;
     158  UINT32             Value;
    165159
    166160  ExtInfo->BitMap = 0;
    167161
    168162  for (Index = 0; Index < Count; Index++) {
    169 
    170163    Opt = Options + Index;
    171164
    172     if (Opt->OptionStr == NULL || Opt->ValueStr == NULL) {
     165    if ((Opt->OptionStr == NULL) || (Opt->ValueStr == NULL)) {
    173166      return EFI_INVALID_PARAMETER;
    174167    }
    175168
    176     if (AsciiStriCmp ((CHAR8 *) Opt->OptionStr, "blksize") == 0) {
     169    if (AsciiStriCmp ((CHAR8 *)Opt->OptionStr, "blksize") == 0) {
    177170      //
    178171      // block size option, valid value is between [8, 65464]
    179172      //
    180       Value = (UINT32) AsciiStrDecimalToUintn ((CHAR8 *) Opt->ValueStr);
     173      Value = (UINT32)AsciiStrDecimalToUintn ((CHAR8 *)Opt->ValueStr);
    181174
    182175      if ((Value < 8) || (Value > 65464)) {
    … …  
    184177      }
    185178
    186       ExtInfo->BlkSize = (UINT16) Value;
     179      ExtInfo->BlkSize = (UINT16)Value;
    187180      ExtInfo->BitMap |= MTFTP6_OPT_BLKSIZE_BIT;
    188 
    189     } else if (AsciiStriCmp ((CHAR8 *) Opt->OptionStr, "timeout") == 0) {
     181    } else if (AsciiStriCmp ((CHAR8 *)Opt->OptionStr, "timeout") == 0) {
    190182      //
    191183      // timeout option, valid value is between [1, 255]
    192184      //
    193       Value = (UINT32) AsciiStrDecimalToUintn ((CHAR8 *) Opt->ValueStr);
    194 
    195       if (Value < 1 || Value > 255) {
     185      Value = (UINT32)AsciiStrDecimalToUintn ((CHAR8 *)Opt->ValueStr);
     186
     187      if ((Value < 1) || (Value > 255)) {
    196188        return EFI_INVALID_PARAMETER;
    197189      }
    198190
    199       ExtInfo->Timeout = (UINT8) Value;
     191      ExtInfo->Timeout = (UINT8)Value;
    200192      ExtInfo->BitMap |= MTFTP6_OPT_TIMEOUT_BIT;
    201 
    202     } else if (AsciiStriCmp ((CHAR8 *) Opt->OptionStr, "tsize") == 0) {
     193    } else if (AsciiStriCmp ((CHAR8 *)Opt->OptionStr, "tsize") == 0) {
    203194      //
    204195      // tsize option, the biggest transfer supported is 4GB with block size option
    205196      //
    206       ExtInfo->Tsize   = (UINT32) AsciiStrDecimalToUintn ((CHAR8 *) Opt->ValueStr);
     197      ExtInfo->Tsize   = (UINT32)AsciiStrDecimalToUintn ((CHAR8 *)Opt->ValueStr);
    207198      ExtInfo->BitMap |= MTFTP6_OPT_TSIZE_BIT;
    208 
    209     } else if (AsciiStriCmp ((CHAR8 *) Opt->OptionStr, "multicast") == 0) {
     199    } else if (AsciiStriCmp ((CHAR8 *)Opt->OptionStr, "multicast") == 0) {
    210200      //
    211201      // Multicast option, if it is a request, the value must be a zero string,
    … …  
    213203      //
    214204      if (!IsRequest) {
    215 
    216205        Status = Mtftp6ParseMcastOption (Opt->ValueStr, ExtInfo);
    217206
    … …  
    220209        }
    221210      } else if (*(Opt->ValueStr) != '\0') {
    222 
    223211        return EFI_INVALID_PARAMETER;
    224212      }
    225213
    226214      ExtInfo->BitMap |= MTFTP6_OPT_MCAST_BIT;
    227 
    228     } else if (AsciiStriCmp ((CHAR8 *) Opt->OptionStr, "windowsize") == 0) {
     215    } else if (AsciiStriCmp ((CHAR8 *)Opt->OptionStr, "windowsize") == 0) {
    229216      if (Operation == EFI_MTFTP6_OPCODE_WRQ) {
    230217        //
    … …  
    234221      }
    235222
    236       Value = (UINT32) AsciiStrDecimalToUintn ((CHAR8 *) Opt->ValueStr);
     223      Value = (UINT32)AsciiStrDecimalToUintn ((CHAR8 *)Opt->ValueStr);
    237224
    238225      if ((Value < 1)) {
    … …  
    240227      }
    241228
    242       ExtInfo->WindowSize = (UINT16) Value;
    243       ExtInfo->BitMap |= MTFTP6_OPT_WINDOWSIZE_BIT;
    244 
     229      ExtInfo->WindowSize = (UINT16)Value;
     230      ExtInfo->BitMap    |= MTFTP6_OPT_WINDOWSIZE_BIT;
    245231    } else if (IsRequest) {
    246232      //
    … …  
    253239  return EFI_SUCCESS;
    254240}
    255 
    256241
    257242/**
    … …  
    274259EFI_STATUS
    275260Mtftp6ParsePacketOption (
    276   IN     EFI_MTFTP6_PACKET     *Packet,
    277   IN     UINT32                PacketLen,
    278   IN OUT UINT32                *Count,
    279   IN     EFI_MTFTP6_OPTION     *Options          OPTIONAL
     261  IN     EFI_MTFTP6_PACKET  *Packet,
     262  IN     UINT32             PacketLen,
     263  IN OUT UINT32             *Count,
     264  IN     EFI_MTFTP6_OPTION  *Options          OPTIONAL
    280265  )
    281266{
    282   UINT8                        *Cur;
    283   UINT8                        *Last;
    284   UINT8                        Num;
    285   UINT8                        *Name;
    286   UINT8                        *Value;
    287 
    288   Num   = 0;
    289   Cur   = (UINT8 *) Packet + MTFTP6_OPCODE_LEN;
    290   Last  = (UINT8 *) Packet + PacketLen - 1;
     267  UINT8  *Cur;
     268  UINT8  *Last;
     269  UINT8  Num;
     270  UINT8  *Name;
     271  UINT8  *Value;
     272
     273  Num  = 0;
     274  Cur  = (UINT8 *)Packet + MTFTP6_OPCODE_LEN;
     275  Last = (UINT8 *)Packet + PacketLen - 1;
    291276
    292277  //
    … …  
    313298    Num++;
    314299
    315     if (Options != NULL && Num <= *Count) {
    316       Options[Num - 1].OptionStr  = Name;
    317       Options[Num - 1].ValueStr   = Value;
     300    if ((Options != NULL) && (Num <= *Count)) {
     301      Options[Num - 1].OptionStr = Name;
     302      Options[Num - 1].ValueStr  = Value;
    318303    }
    319304
    … …  
    324309  // Return buffer too small if the buffer passed-in isn't enough.
    325310  //
    326   if (*Count < Num || Options == NULL) {
     311  if ((*Count < Num) || (Options == NULL)) {
    327312    *Count = Num;
    328313    return EFI_BUFFER_TOO_SMALL;
    … …  
    332317  return EFI_SUCCESS;
    333318}
    334 
    335319
    336320/**
    … …  
    355339EFI_STATUS
    356340Mtftp6ParseStart (
    357   IN     EFI_MTFTP6_PACKET      *Packet,
    358   IN     UINT32                 PacketLen,
    359   IN OUT UINT32                 *OptionCount,
    360      OUT EFI_MTFTP6_OPTION      **OptionList          OPTIONAL
     341  IN     EFI_MTFTP6_PACKET  *Packet,
     342  IN     UINT32             PacketLen,
     343  IN OUT UINT32             *OptionCount,
     344  OUT EFI_MTFTP6_OPTION     **OptionList          OPTIONAL
    361345  )
    362346{
    363   EFI_STATUS                    Status;
    364 
    365   if (PacketLen == 0 || Packet == NULL || OptionCount == NULL) {
     347  EFI_STATUS  Status;
     348
     349  if ((PacketLen == 0) || (Packet == NULL) || (OptionCount == NULL)) {
    366350    return EFI_INVALID_PARAMETER;
    367351  }
    … …  
    380364  // The last byte must be zero to terminate the options.
    381365  //
    382   if (*((UINT8 *) Packet + PacketLen - 1) != 0) {
     366  if (*((UINT8 *)Packet + PacketLen - 1) != 0) {
    383367    return EFI_PROTOCOL_ERROR;
    384368  }
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette