diff options
| author | Rob Mensching <rob@firegiant.com> | 2026-01-30 15:18:43 -0800 |
|---|---|---|
| committer | Rob Mensching <rob@firegiant.com> | 2026-01-30 18:38:41 -0800 |
| commit | 7c3c2cad3c5e95fac151debc89c2f5629b4c6b21 (patch) | |
| tree | 6d43cdcef8d50a2c524b26681418cee425a39651 /src/burn/engine/cabextract.cpp | |
| parent | a16948df3c51f872e750798a967505a93b8439d5 (diff) | |
| download | wix-7c3c2cad3c5e95fac151debc89c2f5629b4c6b21.tar.gz wix-7c3c2cad3c5e95fac151debc89c2f5629b4c6b21.tar.bz2 wix-7c3c2cad3c5e95fac151debc89c2f5629b4c6b21.zip | |
Many small code quality fixes
Diffstat (limited to 'src/burn/engine/cabextract.cpp')
| -rw-r--r-- | src/burn/engine/cabextract.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/burn/engine/cabextract.cpp b/src/burn/engine/cabextract.cpp index 5663c3f7..2140130d 100644 --- a/src/burn/engine/cabextract.cpp +++ b/src/burn/engine/cabextract.cpp | |||
| @@ -833,9 +833,13 @@ static UINT FAR DIAMONDAPI CabWrite( | |||
| 833 | 833 | ||
| 834 | case BURN_CAB_OPERATION_STREAM_TO_BUFFER: | 834 | case BURN_CAB_OPERATION_STREAM_TO_BUFFER: |
| 835 | // copy to target buffer | 835 | // copy to target buffer |
| 836 | memcpy_s(pContext->Cabinet.pbTargetBuffer + pContext->Cabinet.iTargetBuffer, pContext->Cabinet.cbTargetBuffer - pContext->Cabinet.iTargetBuffer, pv, cb); | 836 | if (memcpy_s(pContext->Cabinet.pbTargetBuffer + pContext->Cabinet.iTargetBuffer, pContext->Cabinet.cbTargetBuffer - pContext->Cabinet.iTargetBuffer, pv, cb)) |
| 837 | pContext->Cabinet.iTargetBuffer += cb; | 837 | { |
| 838 | hr = E_INSUFFICIENT_BUFFER; | ||
| 839 | ExitOnRootFailure(hr, "Failed to copy data to target buffer during cabinet extraction."); | ||
| 840 | } | ||
| 838 | 841 | ||
| 842 | pContext->Cabinet.iTargetBuffer += cb; | ||
| 839 | cbWrite = cb; | 843 | cbWrite = cb; |
| 840 | break; | 844 | break; |
| 841 | 845 | ||
