Ayende @ Rahien

Hi!
My name is Oren Eini
Founder of Hibernating Rhinos LTD and RavenDB.
You can reach me by phone or email:

ayende@ayende.com

+972 52-548-6969

, @ Q c

Posts: 10 | Comments: 37

filter by tags archive

Deleting hard links problem

time to read 2 min | 214 words

This code fails:

[DllImport("kernel32.dll", SetLastError = true, CharSet = CharSet.Auto)]
public static extern bool CreateHardLink(string lpFileName, string lpExistingFileName, IntPtr lpSecurityAttributes);

static void Main()
{
    var writer = File.CreateText("test.txt");
    writer.Write("test");
    writer.Flush();

    CreateHardLink("hard_link.txt", "test.txt", IntPtr.Zero);


    File.Delete("hard_link.txt");

}

The problem here is that the hard_link.txt file is considered to be open. What I would expect to happen is that the delete would succeed in removing the directory entry, but keep the file. In other words, I would only expect it to fail if this is the last directory entry for the file (resulting in an actual delete).

For now I changed this to try to delete, but on failure, mark the file to be deletes on the next reboot.


Comments

Steven Kimpe

Hi,

It's probably not "hard_link.txt" that's left open, but rather "text.txt"

This will probably do the trick.

Steven

using(var writer = File.CreateText("test.txt"))

{

writer.Write("test");


writer.Flush();

}

CreateHardLink("hard_link.txt", "test.txt", IntPtr.Zero);

File.Delete("hard_link.txt");

Glenn

Yeah, +1 for Steven. That should work fine.

Ayende Rahien

Steven,

That is pretty much the whole point.

Deleting a hard link isn't actually deleting the file, it is removing the metadata, it should not fail.

Steven Kimpe

Hi Ayende,

I agree completely with you and I find it counter intuitive as well... it's just a different way to tackle the problem.

Bruno Martinez

A better workaround is to open the writer like this

var writer = new StreamWriter(File.Open("test.txt", FileMode.Create, FileAccess.Write, FileShare.Delete));

The downside is that you allow test.txt to be deleted too.

Ayende Rahien

Bruno,

I don't actually control opening that file

Steve Py

Hi Ayende,

I think the trouble is that you're using File.Delete() to try and delete the HardLink when it's really trying you delete the underlying file.

Use the kernel's DeleteFile() method and it should delete the hardlink without the error.

Ayende Rahien

Steve,

Nope, I am trying to delete the hard link, I don't care bout the file.

And File.Delete resolves down to Win32 DeleteFile

Steve Py

Hmm, well it doesn't throw the exception, but I see it doesn't delete the link file directory entry either so long as a handle to the original is in use.

Ayende Rahien

Steve,

Did you check the return code and GetLastError()?

NC
NC

Ayende, are you ok? You haven't posted in 4 days...

marcelo

Ayende, i`ve got that from msdn:

"(...)Flags, attributes, access, and sharing that are specified in CreateFile operate on a per-file basis. That is, if you open a file that does not allow sharing, another application cannot share the file by creating a new hard link to the file.(...)"

src: msdn.microsoft.com/.../aa363860%28VS.85%29.aspx

Comment preview

Comments have been closed on this topic.

FUTURE POSTS

  1. Production postmortem: The case of the memory eater and high load - one day from now
  2. Production postmortem: The case of the lying configuration file - about one day from now
  3. Production postmortem: The industry at large - 3 days from now
  4. The insidious cost of allocations - 4 days from now
  5. Find the bug: The concurrent memory buster - 5 days from now

And 4 more posts are pending...

There are posts all the way to Sep 10, 2015

RECENT SERIES

  1. Find the bug (5):
    20 Apr 2011 - Why do I get a Null Reference Exception?
  2. Production postmortem (10):
    14 Aug 2015 - The case of the man in the middle
  3. What is new in RavenDB 3.5 (7):
    12 Aug 2015 - Monitoring support
  4. Career planning (6):
    24 Jul 2015 - The immortal choices aren't
View all series

Syndication

Main feed Feed Stats
Comments feed   Comments Feed Stats