Changes between Initial Version and Version 1 of Ticket #76, comment 10


Ignore:
Timestamp:
Feb 2, 2020, 4:07:59 PM (5 years ago)
Author:
erdmann

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #76, comment 10

    initial v1  
    11r218:
    22VBoxFS.cpp, "GetEmptyEAs":
    3 the allocated buffer length for pFeal is too small. You have to precompute how large that buffer will by cycling through the pGeal list and computing FEA structure + filename length for each entry and summing up. Once you know that full length you can then allocate the pFeal structure. You will need to ensure that this value does not exceed the initial value of pFEAList->cbList.
     3the allocated buffer length for pFeal is too small. You have to precompute how large that buffer will be by cycling through the pGeal list and computing FEA structure + filename length for each entry and summing up. Once you know that full length you can then allocate the pFeal structure. You will need to ensure that this value does not exceed the initial value of pFEAList->cbList.
    44You might even want to count the number of EAs that you can handle while in the summing up loop described above by always comparing against pFEAList->cbList on each loop iteration.
    5 Once this is all done you can use KernCopyOut to copy your populated FEAList to the original address in user space. You can then even conveniently take the length value that you summed up in your GEA checking loop.
     5Once this is all done you can generate FEAlist and use KernCopyOut to copy your populated FEAList to the original address in user space. You can then even conveniently take the length value that you summed up in your GEA checking loop.