VirtualBox

Changeset 90804 in vbox


Ignore:
Timestamp:
Aug 23, 2021 7:08:53 PM (3 years ago)
Author:
vboxsync
Message:

HostDrivers: More VALID_PTR -> RT_VALID_PTR/AssertPtr.

Location:
trunk/src/VBox/HostDrivers
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/HostDrivers/Support/testcase/tstInt.cpp

    r87235 r90804  
    9999            {
    100100                PVM pVM = CreateVMReq.pVMR3;
    101                 AssertRelease(VALID_PTR(pVM));
     101                AssertRelease(RT_VALID_PTR(pVM));
    102102                AssertRelease(pVM->pVMR0ForCall == CreateVMReq.pVMR0);
    103103                AssertRelease(pVM->pSession == pSession);

  • trunk/src/VBox/HostDrivers/VBoxNetFlt/darwin/VBoxNetFlt-darwin.cpp

    r85344 r90804  
    752752    uint64_t NanoTS = RTTimeSystemNanoTS();
    753753    LogFlow(("vboxNetFltDarwinIffDetached: pThis=%p NanoTS=%RU64 (%d)\n",
    754              pThis, NanoTS, VALID_PTR(pIfNet) ? VBOX_GET_PCOUNT(pIfNet) :  -1));
     754             pThis, NanoTS, RT_VALID_PTR(pIfNet) ? VBOX_GET_PCOUNT(pIfNet) :  -1));
    755755
    756756    Assert(!pThis->fDisconnectedFromHost);
    … …  
    771771
    772772    pIfNet = ASMAtomicUoReadPtrT(&pThis->u.s.pIfNet, ifnet_t);
    773     int cPromisc = VALID_PTR(pIfNet) ? VBOX_GET_PCOUNT(pIfNet) : - 1;
     773    int cPromisc = RT_VALID_PTR(pIfNet) ? VBOX_GET_PCOUNT(pIfNet) : - 1;
    774774
    775775    ASMAtomicUoWriteNullPtr(&pThis->u.s.pIfNet);
    … …  
    836836     * Watch out for the interface going online / offline.
    837837     */
    838     if (    VALID_PTR(pThis)
    839         &&  VALID_PTR(pEvMsg)
    840         &&  pEvMsg->vendor_code  == KEV_VENDOR_APPLE
    841         &&  pEvMsg->kev_class    == KEV_NETWORK_CLASS
    842         &&  pEvMsg->kev_subclass == KEV_DL_SUBCLASS)
     838    if (   RT_VALID_PTR(pThis)
     839        && RT_VALID_PTR(pEvMsg)
     840        && pEvMsg->vendor_code  == KEV_VENDOR_APPLE
     841        && pEvMsg->kev_class    == KEV_NETWORK_CLASS
     842        && pEvMsg->kev_subclass == KEV_DL_SUBCLASS)
    843843    {
    844844        if (pThis->u.s.pIfNet    == pIfNet)
    … …  
    881881        }
    882882        else
    883             Log(("vboxNetFltDarwinIffEvent: pThis->u.s.pIfNet=%p pIfNet=%p (%d)\n", pThis->u.s.pIfNet, pIfNet, VALID_PTR(pIfNet) ? VBOX_GET_PCOUNT(pIfNet) : -1));
    884     }
    885     else if (VALID_PTR(pEvMsg))
     883            Log(("vboxNetFltDarwinIffEvent: pThis->u.s.pIfNet=%p pIfNet=%p (%d)\n", pThis->u.s.pIfNet, pIfNet, RT_VALID_PTR(pIfNet) ? VBOX_GET_PCOUNT(pIfNet) : -1));
     884    }
     885    else if (RT_VALID_PTR(pEvMsg))
    886886        Log(("vboxNetFltDarwinIffEvent: vendor_code=%#x kev_class=%#x kev_subclass=%#x event_code=%#x\n",
    887887             pEvMsg->vendor_code, pEvMsg->kev_class, pEvMsg->kev_subclass, pEvMsg->event_code));

  • trunk/src/VBox/HostDrivers/VBoxNetFlt/linux/VBoxNetFlt-linux.c

    r90249 r90804  
    403403     *       careful here and avoid the race.
    404404     */
    405     if (   !VALID_PTR(pOverride)
     405    if (   !RT_VALID_PTR(pOverride)
    406406        || pOverride->u32Magic != VBOXNETDEVICEOPSOVERRIDE_MAGIC
    407407# if RTLNX_VER_MIN(2,6,29)
    408         || !VALID_PTR(pOverride->pOrgOps)
    409 # endif /* RTLNX_VER_MIN(2,6,29) */
     408        || !RT_VALID_PTR(pOverride->pOrgOps)
     409# endif
    410410        )
    411411    {
    … …  
    427427    pEtherHdr = (PCRTNETETHERHDR)skb_header_pointer(pSkb, 0, cbHdrs, &abHdrBuf[0]);
    428428    if (   pEtherHdr
    429         && VALID_PTR(pOverride->pVBoxNetFlt)
     429        && RT_VALID_PTR(pOverride->pVBoxNetFlt)
    430430        && (pSwitchPort = pOverride->pVBoxNetFlt->pSwitchPort) != NULL
    431         && VALID_PTR(pSwitchPort)
     431        && RT_VALID_PTR(pSwitchPort)
    432432        && cbHdrs >= 6)
    433433    {
    … …  
    458458
    459459    /* Cancel override if ethtool_ops is missing (host-only case, @bugref{5712}) */
    460     if (!VALID_PTR(pDev->OVR_OPS))
     460    if (!RT_VALID_PTR(pDev->OVR_OPS))
    461461        return;
    462462    pOverride = RTMemAlloc(sizeof(*pOverride));
    … …  
    497497    if (!pDev)
    498498        pDev = ASMAtomicUoReadPtrT(&pThis->u.s.pDev, struct net_device *);
    499     if (VALID_PTR(pDev))
     499    if (RT_VALID_PTR(pDev))
    500500    {
    501501        pOverride = (PVBOXNETDEVICEOPSOVERRIDE)pDev->OVR_OPS;
    502         if (    VALID_PTR(pOverride)
    503             &&  pOverride->u32Magic == VBOXNETDEVICEOPSOVERRIDE_MAGIC
    504             &&  VALID_PTR(pOverride->pOrgOps)
     502        if (   RT_VALID_PTR(pOverride)
     503            && pOverride->u32Magic == VBOXNETDEVICEOPSOVERRIDE_MAGIC
     504            && RT_VALID_PTR(pOverride->pOrgOps)
    505505           )
    506506        {

  • trunk/src/VBox/HostDrivers/VBoxNetFlt/solaris/VBoxNetFltBow-solaris.c

    r90785 r90804  
    722722
    723723    PVBOXNETFLTINS pThis = pvArg;
    724     AssertReturnVoid(VALID_PTR(pThis));
    725     AssertReturnVoid(pThis->u.s.hInterface);
     724    AssertPtrReturnVoid(pThis);
     725    AssertPtrReturnVoid(pThis->u.s.hInterface);
    726726
    727727    switch (Type)
    … …  
    14751475     */
    14761476    PVBOXNETFLTVNIC pVNIC = pvIfData;
    1477     AssertMsgReturnVoid(VALID_PTR(pVNIC) && pVNIC->u32Magic == VBOXNETFLTVNIC_MAGIC,
    1478                     ("Invalid pVNIC=%p magic=%#x (expected %#x)\n", pvIfData,
    1479                      VALID_PTR(pVNIC) ? pVNIC->u32Magic : 0, VBOXNETFLTVNIC_MAGIC));
     1477    AssertPtrReturnVoid(pVNIC);
     1478    AssertMsgReturnVoid(pVNIC->u32Magic == VBOXNETFLTVNIC_MAGIC,
     1479                        ("Invalid pVNIC=%p magic=%#x (expected %#x)\n", pvIfData, pVNIC->u32Magic, VBOXNETFLTVNIC_MAGIC));
    14801480    AssertMsgReturnVoid(pVNIC->hLinkId != DATALINK_INVALID_LINKID,
    1481                     ("Invalid hLinkId pVNIC=%p magic=%#x\n", pVNIC, pVNIC->u32Magic));
     1481                        ("Invalid hLinkId pVNIC=%p magic=%#x\n", pVNIC, pVNIC->u32Magic));
    14821482
    14831483    /*

  • trunk/src/VBox/HostDrivers/VBoxUSB/USBFilter.cpp

    r82968 r90804  
    308308USBLIB_DECL(int) USBFilterValidate(PCUSBFILTER pFilter)
    309309{
    310     if (!VALID_PTR(pFilter))
     310    if (!RT_VALID_PTR(pFilter))
    311311        return VERR_INVALID_POINTER;
    312312

  • trunk/src/VBox/HostDrivers/VBoxUSB/VBoxUSBFilterMgr.cpp

    r82968 r90804  
    236236    if (!Owner || Owner == VBOXUSBFILTER_CONTEXT_NIL)
    237237        return VERR_INVALID_PARAMETER;
    238     if (!VALID_PTR(puId))
     238    if (!RT_VALID_PTR(puId))
    239239        return VERR_INVALID_POINTER;
    240240

  • trunk/src/VBox/HostDrivers/VBoxUSB/solaris/VBoxUSB-solaris.c

    r88264 r90804  
    13581358#define CHECKRET_MIN_SIZE(mnemonic, cbMin) \
    13591359    do { \
    1360         if (cbData < (cbMin)) \
     1360        if (RT_UNLIKELY(cbData < (cbMin))) \
    13611361        { \
    13621362            LogRel((DEVICE_NAME ": vboxUsbSolarisProcessIOCtl: " mnemonic ": cbData=%#zx (%zu) min is %#zx (%zu)\n", \
    … …  
    13641364            return VERR_BUFFER_OVERFLOW; \
    13651365        } \
    1366         if ((cbMin) != 0 && !VALID_PTR(pvBuf)) \
     1366        if (RT_UNLIKELY((cbMin) != 0 && !RT_VALID_PTR(pvBuf))) \
    13671367        { \
    13681368            LogRel((DEVICE_NAME ": vboxUsbSolarisProcessIOCtl: " mnemonic ": Invalid pointer %p\n", pvBuf)); \

  • trunk/src/VBox/HostDrivers/VBoxUSB/solaris/VBoxUSBMon-solaris.c

    r82968 r90804  
    578578    }
    579579    if (RT_UNLIKELY(   ReqWrap.cbData != 0
    580                     && !VALID_PTR(pvBuf)))
     580                    && !RT_VALID_PTR(pvBuf)))
    581581    {
    582582        RTMemTmpFree(pvBuf);
    … …  
    658658#define CHECKRET_MIN_SIZE(mnemonic, cbMin) \
    659659    do { \
    660         if (cbData < (cbMin)) \
     660        if (RT_UNLIKELY(cbData < (cbMin))) \
    661661        { \
    662662            LogRel(("vboxUSBSolarisProcessIOCtl: " mnemonic ": cbData=%#zx (%zu) min is %#zx (%zu)\n", \
    … …  
    664664            return VERR_BUFFER_OVERFLOW; \
    665665        } \
    666         if ((cbMin) != 0 && !VALID_PTR(pvData)) \
     666        if (RT_UNLIKELY((cbMin) != 0 && !RT_VALID_PTR(pvData))) \
    667667        { \
    668668            LogRel(("vboxUSBSolarisProcessIOCtl: " mnemonic ": Invalid pointer %p\n", pvData)); \

  • trunk/src/VBox/HostDrivers/VBoxUSB/win/mon/VBoxUsbMon.cpp

    r88531 r90804  
    296296        PDEVICE_RELATIONS pRel = (PDEVICE_RELATIONS)IoStatus.Information;
    297297        LOG(("pRel = %p", pRel));
    298         if (VALID_PTR(pRel))
    299         {
     298        if (RT_VALID_PTR(pRel))
    300299            *pDevRelations = pRel;
    301         }
    302300        else
    303301        {
    … …  
    370368            {
    371369                WCHAR *pId = (WCHAR *)pIoStatus->Information;
    372                 if (VALID_PTR(pId))
     370                if (RT_VALID_PTR(pId))
    373371                {
    374372                    KIRQL Iqrl = KeGetCurrentIrql();
    … …  
    421419                WCHAR *pTmp;
    422420#endif
    423                 if (VALID_PTR(pId))
     421                if (RT_VALID_PTR(pId))
    424422                {
    425423                    KIRQL Iqrl = KeGetCurrentIrql();
    … …  
    582580                        PDEVICE_RELATIONS pRel = (PDEVICE_RELATIONS)pIoStatus->Information;
    583581                        LOG(("pRel = %p", pRel));
    584                         if (VALID_PTR(pRel))
     582                        if (RT_VALID_PTR(pRel))
    585583                        {
    586584                            for (unsigned i=0;i<pRel->Count;i++)
    … …  
    615613            {
    616614                PDEVICE_CAPABILITIES pCaps = pSl->Parameters.DeviceCapabilities.Capabilities;
    617                 if (VALID_PTR(pCaps))
     615                if (RT_VALID_PTR(pCaps))
    618616                {
    619617                    LOG(("Caps.SilentInstall  = %d", pCaps->SilentInstall));
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