Menu

#163 Perforce versioning system fails when running sync command o

v1.0 (example)
open-fixed
nobody
8
2015-07-09
2013-03-15
No

Perforce versioning system fails when running sync command on hgfs filesytem.

The problem is that perforce client uses rename operation upon a temp file which has read-only attributes. The operation then fails with permission denied (see attached test application).

The fix we have provided removes the read only attribute and retries the rename operation again (see attached source code and diff). The same code is already there (see function HgfsDelete) – probably it is a good idea to refactor the retry procedure.

Could you please add this fix to the upstream of the VMware tools?
Thanks a lot,
David.

Discussion

  • David Koudela

    David Koudela - 2013-03-15

    Fixed Source Code

     
  • David Koudela

    David Koudela - 2013-03-15

    Diff

     
  • David Koudela

    David Koudela - 2013-03-15

    Test Application

     
  • David Koudela

    David Koudela - 2013-03-15
    • priority: 5 --> 8
     
  • Dmitry Torokhov

    Dmitry Torokhov - 2013-03-15

    Thank you David, I'll forward this on.

     
  • Steve

    Steve - 2013-05-06

    Hi David,
    I currently work on the HGFS file system, and just got notice of this from dtor.

    Thanks for looking into this, I will take a look at the modified file and run it through some checks and tests.
    I recently ran some perforce sync in my Ubuntu VM running on my Windows 7 host.
    I didn't see any issues of note, but I will retry and pay close attention.

     
  • Steve

    Steve - 2013-05-06

    David, I have reproduced this issue and will apply your diffs and yes, probably refactor since HgfsDelete does the same thing.

    Thanks.
    Steve

     
  • David Koudela

    David Koudela - 2013-05-07

    Hi Steve,
    Thanks a lot!
    David.

     
  • David Koudela

    David Koudela - 2013-09-11

    Hi Steve,
    Is there any update?
    I see that VMware Workstation 10 has been recently released.
    Is the fix included in this release?
    Thanks,
    David.

     
  • David Koudela

    David Koudela - 2014-04-03

    Any update?

     
    • Ravindra Kumar

      Ravindra Kumar - 2014-04-11

      Thanks for the patch David. I would like to confirm that your patch has been included in the upstream and you will see it in the next release of open-vm-tools.

       
  • Ravindra Kumar

    Ravindra Kumar - 2015-07-09

    Please check with the latest release of open-vm-tools 9.10.2.

     
  • Ravindra Kumar

    Ravindra Kumar - 2015-07-09
    • status: open --> open-fixed
    • Group: --> v1.0 (example)
     

Log in to post a comment.