VirtualBox

Changeset 90815 in vbox for trunk/src


Ignore:
Timestamp:
Aug 23, 2021 8:17:22 PM (3 years ago)
Author:
vboxsync
Message:

Storage: More VALID_PTR -> RT_VALID_PTR/AssertPtr.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Storage/VD.cpp

    r90814 r90815  
    708708DECLINLINE(void) vdIoCtxInit(PVDIOCTX pIoCtx, PVDISK pDisk, VDIOCTXTXDIR enmTxDir,
    709709                             uint64_t uOffset, size_t cbTransfer, PVDIMAGE pImageStart,
    710                              PCRTSGBUF pcSgBuf, void *pvAllocation,
     710                             PCRTSGBUF pSgBuf, void *pvAllocation,
    711711                             PFNVDIOCTXTRANSFER pfnIoCtxTransfer, uint32_t fFlags)
    712712{
    … …  
    735735    if (   enmTxDir != VDIOCTXTXDIR_FLUSH
    736736        && enmTxDir != VDIOCTXTXDIR_DISCARD)
    737         RTSgBufClone(&pIoCtx->Req.Io.SgBuf, pcSgBuf);
     737        RTSgBufClone(&pIoCtx->Req.Io.SgBuf, pSgBuf);
    738738    else
    739739        memset(&pIoCtx->Req.Io.SgBuf, 0, sizeof(RTSGBUF));
    … …  
    10091009DECLINLINE(PVDIOCTX) vdIoCtxAlloc(PVDISK pDisk, VDIOCTXTXDIR enmTxDir,
    10101010                                  uint64_t uOffset, size_t cbTransfer,
    1011                                   PVDIMAGE pImageStart,PCRTSGBUF pcSgBuf,
     1011                                  PVDIMAGE pImageStart,PCRTSGBUF pSgBuf,
    10121012                                  void *pvAllocation, PFNVDIOCTXTRANSFER pfnIoCtxTransfer,
    10131013                                  uint32_t fFlags)
    … …  
    10191019    {
    10201020        vdIoCtxInit(pIoCtx, pDisk, enmTxDir, uOffset, cbTransfer, pImageStart,
    1021                     pcSgBuf, pvAllocation, pfnIoCtxTransfer, fFlags);
     1021                    pSgBuf, pvAllocation, pfnIoCtxTransfer, fFlags);
    10221022    }
    10231023
    … …  
    10271027DECLINLINE(PVDIOCTX) vdIoCtxRootAlloc(PVDISK pDisk, VDIOCTXTXDIR enmTxDir,
    10281028                                      uint64_t uOffset, size_t cbTransfer,
    1029                                       PVDIMAGE pImageStart, PCRTSGBUF pcSgBuf,
     1029                                      PVDIMAGE pImageStart, PCRTSGBUF pSgBuf,
    10301030                                      PFNVDASYNCTRANSFERCOMPLETE pfnComplete,
    10311031                                      void *pvUser1, void *pvUser2,
    … …  
    10351035{
    10361036    PVDIOCTX pIoCtx = vdIoCtxAlloc(pDisk, enmTxDir, uOffset, cbTransfer, pImageStart,
    1037                                    pcSgBuf, pvAllocation, pfnIoCtxTransfer, fFlags);
     1037                                   pSgBuf, pvAllocation, pfnIoCtxTransfer, fFlags);
    10381038
    10391039    if (RT_LIKELY(pIoCtx))
    … …  
    11011101DECLINLINE(PVDIOCTX) vdIoCtxChildAlloc(PVDISK pDisk, VDIOCTXTXDIR enmTxDir,
    11021102                                       uint64_t uOffset, size_t cbTransfer,
    1103                                        PVDIMAGE pImageStart, PCRTSGBUF pcSgBuf,
     1103                                       PVDIMAGE pImageStart, PCRTSGBUF pSgBuf,
    11041104                                       PVDIOCTX pIoCtxParent, size_t cbTransferParent,
    11051105                                       size_t cbWriteParent, void *pvAllocation,
    … …  
    11071107{
    11081108    PVDIOCTX pIoCtx = vdIoCtxAlloc(pDisk, enmTxDir, uOffset, cbTransfer, pImageStart,
    1109                                    pcSgBuf, pvAllocation, pfnIoCtxTransfer, pIoCtxParent->fFlags & ~VDIOCTX_FLAGS_DONT_FREE);
     1109                                   pSgBuf, pvAllocation, pfnIoCtxTransfer, pIoCtxParent->fFlags & ~VDIOCTX_FLAGS_DONT_FREE);
    11101110
    11111111    AssertPtr(pIoCtxParent);
    … …  
    94839483
    94849484VBOXDDU_DECL(int) VDAsyncRead(PVDISK pDisk, uint64_t uOffset, size_t cbRead,
    9485                               PCRTSGBUF pcSgBuf,
     9485                              PCRTSGBUF pSgBuf,
    94869486                              PFNVDASYNCTRANSFERCOMPLETE pfnComplete,
    94879487                              void *pvUser1, void *pvUser2)
    … …  
    94919491    PVDIOCTX pIoCtx = NULL;
    94929492
    9493     LogFlowFunc(("pDisk=%#p uOffset=%llu pcSgBuf=%#p cbRead=%zu pvUser1=%#p pvUser2=%#p\n",
    9494                  pDisk, uOffset, pcSgBuf, cbRead, pvUser1, pvUser2));
     9493    LogFlowFunc(("pDisk=%#p uOffset=%llu pSgBuf=%#p cbRead=%zu pvUser1=%#p pvUser2=%#p\n",
     9494                 pDisk, uOffset, pSgBuf, cbRead, pvUser1, pvUser2));
    94959495
    94969496    /* sanity check */
    … …  
    95009500    /* Check arguments. */
    95019501    AssertReturn(cbRead > 0, VERR_INVALID_PARAMETER);
    9502     AssertPtrReturn(pcSgBuf, VERR_INVALID_POINTER);
     9502    AssertPtrReturn(pSgBuf, VERR_INVALID_POINTER);
    95039503
    95049504    do
    … …  
    95159515
    95169516        pIoCtx = vdIoCtxRootAlloc(pDisk, VDIOCTXTXDIR_READ, uOffset,
    9517                                   cbRead, pDisk->pLast, pcSgBuf,
     9517                                  cbRead, pDisk->pLast, pSgBuf,
    95189518                                  pfnComplete, pvUser1, pvUser2,
    95199519                                  NULL, vdReadHelperAsync,
    … …  
    95509550
    95519551VBOXDDU_DECL(int) VDAsyncWrite(PVDISK pDisk, uint64_t uOffset, size_t cbWrite,
    9552                                PCRTSGBUF pcSgBuf,
     9552                               PCRTSGBUF pSgBuf,
    95539553                               PFNVDASYNCTRANSFERCOMPLETE pfnComplete,
    95549554                               void *pvUser1, void *pvUser2)
    … …  
    95569556    int rc;
    95579557    int rc2;
    9558     bool fLockWrite = false;
    95599558    PVDIOCTX pIoCtx = NULL;
    95609559
    9561     LogFlowFunc(("pDisk=%#p uOffset=%llu cSgBuf=%#p cbWrite=%zu pvUser1=%#p pvUser2=%#p\n",
    9562                  pDisk, uOffset, pcSgBuf, cbWrite, pvUser1, pvUser2));
     9560    LogFlowFunc(("pDisk=%#p uOffset=%llu pSgBuf=%#p cbWrite=%zu pvUser1=%#p pvUser2=%#p\n",
     9561                 pDisk, uOffset, pSgBuf, cbWrite, pvUser1, pvUser2));
     9562    /* sanity check */
     9563    AssertPtrReturn(pDisk, VERR_INVALID_POINTER);
     9564    AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
     9565
     9566    /* Check arguments. */
     9567    AssertReturn(cbWrite > 0, VERR_INVALID_PARAMETER);
     9568    AssertPtrReturn(pSgBuf, VERR_INVALID_POINTER);
     9569
    95639570    do
    95649571    {
    9565         /* sanity check */
    9566         AssertPtrBreakStmt(pDisk, rc = VERR_INVALID_PARAMETER);
    9567         AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
    9568 
    9569         /* Check arguments. */
    9570         AssertMsgBreakStmt(cbWrite,
    9571                            ("cbWrite=%zu\n", cbWrite),
    9572                            rc = VERR_INVALID_PARAMETER);
    9573         AssertMsgBreakStmt(VALID_PTR(pcSgBuf),
    9574                            ("pcSgBuf=%#p\n", pcSgBuf),
    9575                            rc = VERR_INVALID_PARAMETER);
    9576 
    95779572        rc2 = vdThreadStartWrite(pDisk);
    95789573        AssertRC(rc2);
    9579         fLockWrite = true;
    95809574
    95819575        AssertMsgBreakStmt(   uOffset < pDisk->cbSize
    … …  
    95879581
    95889582        pIoCtx = vdIoCtxRootAlloc(pDisk, VDIOCTXTXDIR_WRITE, uOffset,
    9589                                   cbWrite, pDisk->pLast, pcSgBuf,
     9583                                  cbWrite, pDisk->pLast, pSgBuf,
    95909584                                  pfnComplete, pvUser1, pvUser2,
    95919585                                  NULL, vdWriteHelperAsync,
    … …  
    96099603    } while (0);
    96109604
    9611     if (RT_UNLIKELY(fLockWrite) && (rc != VERR_VD_ASYNC_IO_IN_PROGRESS))
     9605    if (rc != VERR_VD_ASYNC_IO_IN_PROGRESS)
    96129606    {
    96139607        rc2 = vdThreadFinishWrite(pDisk);
    … …  
    96259619    int rc;
    96269620    int rc2;
    9627     bool fLockWrite = false;
    96289621    PVDIOCTX pIoCtx = NULL;
    96299622
    96309623    LogFlowFunc(("pDisk=%#p\n", pDisk));
     9624    /* sanity check */
     9625    AssertPtrReturn(pDisk, VERR_INVALID_POINTER);
     9626    AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
    96319627
    96329628    do
    96339629    {
    9634         /* sanity check */
    9635         AssertPtrBreakStmt(pDisk, rc = VERR_INVALID_PARAMETER);
    9636         AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
    9637 
    96389630        rc2 = vdThreadStartWrite(pDisk);
    96399631        AssertRC(rc2);
    9640         fLockWrite = true;
    96419632
    96429633        AssertPtrBreakStmt(pDisk->pLast, rc = VERR_VD_NOT_OPENED);
    … …  
    96659656    } while (0);
    96669657
    9667     if (RT_UNLIKELY(fLockWrite) && (rc != VERR_VD_ASYNC_IO_IN_PROGRESS))
     9658    if (rc != VERR_VD_ASYNC_IO_IN_PROGRESS)
    96689659    {
    96699660        rc2 = vdThreadFinishWrite(pDisk);
    … …  
    96819672    int rc;
    96829673    int rc2;
    9683     bool fLockWrite = false;
    96849674    PVDIOCTX pIoCtx = NULL;
    96859675
    96869676    LogFlowFunc(("pDisk=%#p\n", pDisk));
     9677    /* sanity check */
     9678    AssertPtrReturn(pDisk, VERR_INVALID_POINTER);
     9679    AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
    96879680
    96889681    do
    96899682    {
    9690         /* sanity check */
    9691         AssertPtrBreakStmt(pDisk, rc = VERR_INVALID_PARAMETER);
    9692         AssertMsg(pDisk->u32Signature == VDISK_SIGNATURE, ("u32Signature=%08x\n", pDisk->u32Signature));
    9693 
    96949683        rc2 = vdThreadStartWrite(pDisk);
    96959684        AssertRC(rc2);
    9696         fLockWrite = true;
    96979685
    96989686        AssertPtrBreakStmt(pDisk->pLast, rc = VERR_VD_NOT_OPENED);
    … …  
    97209708    } while (0);
    97219709
    9722     if (RT_UNLIKELY(fLockWrite) && (rc != VERR_VD_ASYNC_IO_IN_PROGRESS))
     9710    if (rc != VERR_VD_ASYNC_IO_IN_PROGRESS)
    97239711    {
    97249712        rc2 = vdThreadFinishWrite(pDisk);
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