extattr 0.1 → 0.1.1

data/README.txt

@@ -1,53 +1,73 @@
 = extattr
 
-extattr is extended attribute operation library for Windows and FreeBSD.
+extattr is filesystem extended attributes operation library for FreeBSD, GNU/Linux and Microsoft Windows.
 
 ----
 
-extattr は拡張属性を操作するライブラリで、Windows、FreeBSD に対応しています (GNU/Linux はコードを書いてみただけで、試験がまったくされていません)。
+extattr はファイルシステムの拡張属性を操作するライブラリで、FreeBSD、GNU/Linux、Windows に対応しています。
 
 サポートされる環境で、統一的なメソッドを提供します。
 
-クラスメソッドに『!』がついているものはシンボリックリンクに対する操作となります。
 
-キーワード引数の <code>namespace</code> を与えると、拡張属性の名前空間を指定できます。規定値は <code>EXTATTR_NAMESPACE_USER</code> で、ほかの値は <code>EXTATTR_NAMESPACE_SYSTEM</code> のみが指定できます (Windows 版では <code>EXTATTR_NAMESPACE_USER</code> のみが指定可能です)。
+== test system
+
+- Microsoft Windows XP Professional SP3
+- PC-BSD/AMD64 9.0
+- lubuntu 12.04
+
+
+== 簡易リファレンスマニュアル
+
+クラスメソッドに『!』がついているものはシンボリックリンクに対する操作となります。
 
+キーワード引数の <code>namespace</code> を与えると、拡張属性の名前空間を指定できます。
+規定値は <code>EXTATTR_NAMESPACE_USER</code> で、ほかの値は <code>EXTATTR_NAMESPACE_SYSTEM</code> のみが指定できます
+(Windows 版では <code>EXTATTR_NAMESPACE_USER</code> のみが指定可能です)。
 
-== 拡張属性の属性名を取得:
+拡張属性の属性名を取得:
 
-- File#extattr_list(namespace: File::EXTATTR_NAMESPACE_USER) -> array
-- File.extattr_list(path, namespace: File::EXTATTR_NAMESPACE_USER) -> array
-- File.extattr_list!(path, namespace: File::EXTATTR_NAMESPACE_USER) -> array
+    File#extattr_list(namespace: File::EXTATTR_NAMESPACE_USER) -> array
+    File#extattr_list(namespace: File::EXTATTR_NAMESPACE_USER) { |name| ... } -> nil
+    File.extattr_list(path, namespace: File::EXTATTR_NAMESPACE_USER) -> array
+    File.extattr_list(path, namespace: File::EXTATTR_NAMESPACE_USER) { |name| ... } -> nil
+    File.extattr_list!(path, namespace: File::EXTATTR_NAMESPACE_USER) -> array
+    File.extattr_list!(path, namespace: File::EXTATTR_NAMESPACE_USER) { |name| ... } -> nil
 
-== 拡張属性の要素の大きさを取得:
+拡張属性の要素の大きさを取得:
 
-- File#extattr_size(name, namespace: File::EXTATTR_NAMESPACE_USER) -> size
-- File.extattr_size(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> size
-- File.extattr_size!(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> size
+    File#extattr_size(name, namespace: File::EXTATTR_NAMESPACE_USER) -> size
+    File.extattr_size(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> size
+    File.extattr_size!(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> size
 
-== 拡張属性の要素を取得:
+拡張属性の要素を取得:
 
-- File#extattr_get(name, namespace: File::EXTATTR_NAMESPACE_USER) -> data (String)
-- File.extattr_get(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> data (String)
-- File.extattr_get!(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> data (String)
+    File#extattr_get(name, namespace: File::EXTATTR_NAMESPACE_USER) -> data (String)
+    File.extattr_get(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> data (String)
+    File.extattr_get!(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> data (String)
 
-== 拡張属性の要素を設定:
+拡張属性の要素を設定:
 
-- File#extattr_set(name, data, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
-- File.extattr_set(path, name, data, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
-- File.extattr_set!(path, name, data, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
+    File#extattr_set(name, data, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
+    File.extattr_set(path, name, data, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
+    File.extattr_set!(path, name, data, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
 
-== 拡張属性の要素を削除:
+拡張属性の要素を削除:
 
-- File#extattr_delete(name, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
-- File.extattr_delete(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
-- File.extattr_delete!(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
+    File#extattr_delete(name, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
+    File.extattr_delete(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
+    File.extattr_delete!(path, name, namespace: File::EXTATTR_NAMESPACE_USER) -> nil
 
 
 == Microsoft Windows における諸注意
 
 Windows 2000 以降でのみ動作します。Windows 9X シリーズでは <code>require "extattr"</code> の段階で例外が発生するでしょう。
 
-リパースポイント (ジャンクションやシンボリックリンク) に対する ADS は要素の取得や設定、削除は出来ません。必ずリンク先に対する操作となります。
+リパースポイント (ジャンクションやシンボリックリンク) に対する ADS は要素の取得や設定、削除は出来ません。
+必ずリンク先に対する操作となります。
 
-128 KiB を超える ADS は取得も設定も出来ません。これは『拡張属性』と捉えた場合、巨大なデータを扱えるべきではないという考えによるためです (本当のところは FreeBSD の拡張属性が最大 64KiB 弱であることが由来です)。巨大な ADS を扱いたい場合は、File.open で自由に読み書きできます (これは ruby に限ったことではなく、Windows による仕様です)。この場合の与えるファイル名は、path + ":" + name という形になります。
+64 KiB を超える ADS は取得も設定も出来ません。
+これは『拡張属性』と捉えた場合、巨大なデータを扱えるべきではないという考えによるためです
+(本当のところは FreeBSD の拡張属性が最大 64KiB 弱であることが由来です)。
+巨大な ADS を扱いたい場合は、<code>File.open</code> でファイルとして扱えるので自由に読み書きできます
+(これは ruby に限ったことではなく、Windows による仕様です)。
+この場合の与えるファイル名は、<code>path + ":" + name</code> という形になります。

data/ext/extattr.c

@@ -61,10 +61,10 @@ file2fd(VALUE file)
 
 #if defined(HAVE_SYS_EXTATTR_H)
 #   include "extattr.bsd"
-#elif defined(HAVE_ATTR_XATTR_H)
-#   include "extattr.linux"
 #elif defined(HAVE_WINNT_H)
 #   include "extattr.windows"
+#elif defined(HAVE_ATTR_XATTR_H) || defined(HAVE_SYS_XATTR_H)
+#   include "extattr.linux"
 #else
 #   error ruby extattr not supported on your system
 #endif

data/ext/extattr.linux

@@ -1,33 +1,39 @@
 #include <sys/types.h>
-#include <attr/xattr.h>
+#if HAVE_ATTR_XATTR_H
+#   include <attr/xattr.h>
+#else
+#   include <sys/xattr.h>
+#endif
+
+enum {
+    EXTATTR_NAMESPACE_USER,
+    EXTATTR_NAMESPACE_SYSTEM,
+    EXTATTR_NAMESPACE_TRUSTED,
+    EXTATTR_NAMESPACE_SECURITY,
+};
 
-#define EXTATTR_NAMESPACE_USER 0
-#define EXTATTR_NAMESPACE_SYSTEM 1
 
 static VALUE NAMESPACE_USER_PREFIX, NAMESPACE_SYSTEM_PREFIX;
 
 
 static VALUE
-extattr_list0(ssize_t (*func)(), void *d, int namespace)
+xattr_name(int namespace, VALUE name)
 {
-    ssize_t size = 65536;
-    VALUE buf = rb_str_buf_new(size);
-    char *ptr = RSTRING_PTR(buf);
-    size = func(d, ptr, size);
-    if (size < 0) { rb_sys_fail("listxattr call error"); }
-
-    VALUE (*reduce)(void *, VALUE);
-    void *first;
-    VALUE list;
-    if (rb_block_given_p()) {
-        reduce = (VALUE (*)(void *, VALUE))rb_yield_values;
-        first = (void *)1;
-        list = Qnil;
-    } else {
-        reduce = (VALUE (*)(void *, VALUE))rb_ary_push;
-        first = (void *)(list = rb_ary_new());
+    switch (namespace) {
+    case EXTATTR_NAMESPACE_USER:
+        return rb_str_plus(NAMESPACE_USER_PREFIX, name);
+    case EXTATTR_NAMESPACE_SYSTEM:
+        return rb_str_plus(NAMESPACE_SYSTEM_PREFIX, name);
+    default:
+        rb_raise(rb_eRuntimeError, "namespace error");
+        return Qnil;
     }
+}
+
 
+static inline void
+extattr_list_name(const char *ptr, size_t size, int namespace, VALUE (*reduce)(void *, VALUE), void *user)
+{
     const char *end = ptr + size;
     while (ptr < end) {
         int len = strlen(ptr);
@@ -42,11 +48,32 @@ extattr_list0(ssize_t (*func)(), void *d, int namespace)
         }
 
         name = rb_str_new_cstr(ptr);
-        reduce(first, name);
+        reduce(user, name);
         ptr += RSTRING_LEN(name) + 1; // 最後の『+1』は、ヌルバイトの分。
     }
+}
+
+static VALUE
+extattr_list0(ssize_t (*func)(), void *d, int namespace)
+{
+    ssize_t size = 65536;
+    VALUE buf = rb_str_buf_new(size);
+    char *ptr = RSTRING_PTR(buf);
+    size = func(d, ptr, size);
+    if (size < 0) { rb_sys_fail("listxattr call error"); }
 
-    return list;
+    if (rb_block_given_p()) {
+        extattr_list_name(ptr, size, namespace,
+                          (VALUE (*)(void *, VALUE))rb_yield_values,
+                          (void *)1);
+        return Qnil;
+    } else {
+        VALUE list = rb_ary_new();
+        extattr_list_name(ptr, size, namespace,
+                          (VALUE (*)(void *, VALUE))rb_ary_push,
+                          (void *)list);
+        return list;
+    }
 }
 
 static VALUE
@@ -68,20 +95,6 @@ file_s_extattr_list_link0(VALUE path, int namespace)
 }
 
 
-static VALUE
-xattr_name(int namespace, VALUE name)
-{
-    switch (namespace) {
-    case EXTATTR_NAMESPACE_USER:
-        return rb_str_plus(NAMESPACE_USER_PREFIX, name);
-    case EXTATTR_NAMESPACE_SYSTEM:
-        return rb_str_plus(NAMESPACE_SYSTEM_PREFIX, name);
-    default:
-        rb_raise(rb_eRuntimeError, "namespace error");
-        return Qnil;
-    }
-}
-
 static VALUE
 extattr_size0(ssize_t (*func)(), void *d, int namespace, VALUE name)
 {
@@ -146,7 +159,7 @@ static VALUE
 extattr_set0(int (*func)(), void *d, int namespace, VALUE name, VALUE data)
 {
     name = xattr_name(namespace, name);
-    int status = func(d, StringValueCStr(name), RSTRING_PTR(data), RSTRING_LEN(data));
+    int status = func(d, StringValueCStr(name), RSTRING_PTR(data), RSTRING_LEN(data), 0);
     if (status < 0) { rb_sys_fail("getxattr call error"); }
     return Qnil;
 }

data/ext/extattr.windows

@@ -6,24 +6,27 @@
 #include <ntdef.h>
 #include <psapi.h>
 #include <wchar.h>
+#include <ruby/win32.h>
 
 #define EXTATTR_NAMESPACE_USER 0
 #define EXTATTR_NAMESPACE_SYSTEM 1
 
 /*
- * ADS は普通のファイルのように扱えるから、extattr とみなして扱う場合は容量の制限を設けることにする。
- * ruby-extattr で不十分な巨大なデータを扱う場合は File.open で開くことが出来るので必要であればそちらで。
- * TODO: 最適値を探すべき
+ * - TODO: バッファサイズの確保量が結構大きめ。最適化をするべき。
+ * - TODO: CloseHandle してないところがありすぎ。修正。
+ * - TODO: Win9X シリーズだとこのライブラリ読み込みすら出来ない。
+ *   メソッド定義だけは行うようにするべき? (実際に呼び出したら NotImplementedError 例外を投げる、とか)
+ * - リパースポイントって、ADSはつけられないのかな? 操作方法がわからない。
  */
-static const size_t EXTATTR_MAX = 131072;
 
 /*
- * TODO: バッファサイズの確保量が結構大きめ。最適化をするべき。
- * TODO: CloseHandle してないところがありすぎ。修正。
- * TODO: Win9X シリーズだとこのライブラリ読み込みすら出来ない。
- *       メソッド定義だけは行うようにするべき? (実際に呼び出したら NotImpError 例外を投げる)
- * リパースポイントって、ADSはつけられないのかな? 操作方法がわからない。
+ * ADS は普通のファイルのように扱えるから、extattr とみなして扱う場合は容量の制限を設けることにする。
+ *
+ * ruby-extattr で不十分な巨大なデータを扱う場合は File.open で開くことが出来るので必要であればそちらで。
+ *
+ * TODO: 最適値を探すべき
  */
+static const size_t EXTATTR_MAX = 65536;
 
 
 static rb_encoding *ENCutf8p;
@@ -32,8 +35,6 @@ static rb_encoding *ENCutf16lep;
 static VALUE ENCutf16le;
 
 
-
-
 static VALUE
 str2wcs(VALUE str)
 {
@@ -90,7 +91,7 @@ wpath2str(const wchar_t path[], size_t size)
 static VALUE
 adsname2str(const wchar_t name[], size_t size)
 {
-    if (name[0] == L':' && wcsncmp(name + size - 6, L":$DATA", 6) == 0) {
+    if (name[0] == L':' && size >= 7 && wcsncmp(name + size - 6, L":$DATA", 6) == 0) {
         name += 1;
         size -= 7;
     }
@@ -100,9 +101,15 @@ adsname2str(const wchar_t name[], size_t size)
 
 
 static void
-raise_win32_error(DWORD status)
+raise_win32_error(DWORD status, VALUE name)
 {
-    rb_raise(rb_eSystemCallError, "Win32 API error - %u (%08Xh)", status, status);
+    if (NIL_P(name)) {
+        VALUE args[] = { INT2NUM(rb_w32_map_errno(status)), };
+        rb_exc_raise(rb_class_new_instance(1, args, rb_eSystemCallError));
+    } else {
+        VALUE args[] = { StringValue(name), INT2NUM(rb_w32_map_errno(status)), };
+        rb_exc_raise(rb_class_new_instance(2, args, rb_eSystemCallError));
+    }
 }
 
 static void
@@ -111,6 +118,15 @@ raise_ntstatus_error(NTSTATUS status)
     rb_raise(rb_eSystemCallError, "NTSTATUS error - %u (%08Xh)", status, status);
 }
 
+static void
+check_status_error(NTSTATUS status)
+{
+    if (status != STATUS_SUCCESS) {
+        raise_ntstatus_error(status);
+    }
+}
+
+
 static VALUE
 get_filepath(HANDLE file)
 {
@@ -121,7 +137,8 @@ get_filepath(HANDLE file)
                                     RSTRING_PTR(buf),
                                     size,
                                     &size);
-    if (status != 0) { raise_ntstatus_error(status); }
+    check_status_error(status);
+
     const OBJECT_NAME_INFORMATION *info = (const OBJECT_NAME_INFORMATION *)RSTRING_PTR(buf);
     if (wcsnicmp(info->Name.Buffer, L"\\Device\\", 8) == 0) {
         // 先頭の "/Device/" を "//?/" に置き換える
@@ -138,7 +155,7 @@ get_filesize(HANDLE file)
     BY_HANDLE_FILE_INFORMATION info;
 
     if (!GetFileInformationByHandle(file, &info)) {
-        raise_win32_error(GetLastError());
+        raise_win32_error(GetLastError(), Qnil);
     }
 
     return ((uint64_t)info.nFileSizeHigh << 32) | info.nFileSizeLow;
@@ -163,35 +180,47 @@ check_namespace(int namespace)
 }
 
 
-static VALUE
-extattr_list0(HANDLE file)
+static void
+extattr_list_name(const char *ptr, void *(*yield)(void *user, VALUE name), void *user)
 {
-    VALUE iostatusblock = rb_str_buf_new(4096);
-    VALUE infobuf = rb_str_buf_new(65536);
-    rb_str_set_len(infobuf, 65536);
-    NTSTATUS status = NtQueryInformationFile(file,
-                                             (PIO_STATUS_BLOCK)RSTRING_PTR(iostatusblock),
-                                             RSTRING_PTR(infobuf), RSTRING_LEN(infobuf),
-                                             FileStreamInformation);
-    if (status != 0) { raise_ntstatus_error(status); }
-    const char *ptr = RSTRING_PTR(infobuf);
-    VALUE list = Qnil;
-    if (!rb_block_given_p()) { list = rb_ary_new(); }
     for (;;) {
         const FILE_STREAM_INFORMATION *info = (const FILE_STREAM_INFORMATION *)ptr;
         VALUE name = adsname2str(info->StreamName, info->StreamNameLength / 2);
         if (RSTRING_LEN(name) > 0) {
-            if (NIL_P(list)) {
-                rb_yield(name);
-            } else {
-                rb_ary_push(list, name);
-            }
+            yield(user, name);
         }
         size_t size = info->NextEntryOffset;
         if (size == 0) { break; }
         ptr += size;
     }
-    return list;
+}
+
+static VALUE
+extattr_list0(HANDLE file)
+{
+    VALUE iostatusblock = rb_str_buf_new(4096);
+    size_t size = 65536; // TODO: 最適値を見つける
+    VALUE infobuf = rb_str_buf_new(size);
+    char *ptr = RSTRING_PTR(infobuf);
+    memset(ptr, 0, sizeof(FILE_STREAM_INFORMATION));
+    NTSTATUS status = NtQueryInformationFile(file,
+                                             (PIO_STATUS_BLOCK)RSTRING_PTR(iostatusblock),
+                                             ptr, size,
+                                             FileStreamInformation);
+    check_status_error(status);
+
+    if (rb_block_given_p()) {
+        extattr_list_name(ptr,
+                          (void *(*)(void *, VALUE))rb_yield_values,
+                          (void *)1);
+        return Qnil;
+    } else {
+        VALUE list = rb_ary_new();
+        extattr_list_name(ptr,
+                          (void *(*)(void *, VALUE))rb_ary_push,
+                          (void *)list);
+        return list;
+    }
 }
 
 static VALUE
@@ -242,12 +271,13 @@ static VALUE
 file_s_extattr_list0(VALUE path, int namespace)
 {
     check_namespace(namespace);
-    HANDLE file = CreateFileW(STR2WPATH(path), 0,
+    VALUE path1 = path;
+    HANDLE file = CreateFileW(STR2WPATH(path1), 0,
                               FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
                               NULL, 3,
                               FILE_FLAG_BACKUP_SEMANTICS,
                               0);
-    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError()); }
+    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError(), path); }
     return rb_ensure(extattr_list0, (VALUE)file, file_close, (VALUE)file);
 }
 
@@ -255,12 +285,13 @@ static VALUE
 file_s_extattr_list_link0(VALUE path, int namespace)
 {
     check_namespace(namespace);
-    HANDLE file = CreateFileW(STR2WPATH(path), 0,
+    VALUE path1 = path;
+    HANDLE file = CreateFileW(STR2WPATH(path1), 0,
                               FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
                               NULL, 3,
                               FILE_FLAG_BACKUP_SEMANTICS | FILE_FLAG_OPEN_REPARSE_POINT,
                               0);
-    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError()); }
+    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError(), path); }
     return rb_ensure(extattr_list0, (VALUE)file, file_close, (VALUE)file);
 }
 
@@ -268,14 +299,14 @@ static VALUE
 file_s_extattr_size0(VALUE path, int namespace, VALUE name)
 {
     check_namespace(namespace);
-    path = rb_str_plus(path, rb_str_new(":", 1));
-    rb_str_append(path, name);
-    HANDLE file = CreateFileW(STR2WPATH(path), 0,
+    VALUE path1 = rb_str_plus(path, rb_str_new(":", 1));
+    rb_str_append(path1, name);
+    HANDLE file = CreateFileW(STR2WPATH(path1), 0,
                               FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
                               NULL, 3,
                               FILE_FLAG_BACKUP_SEMANTICS,
                               0);
-    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError()); }
+    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError(), path); }
     return rb_ensure(extattr_size0, (VALUE)file, file_close, (VALUE)file);
 }
 
@@ -283,14 +314,14 @@ static VALUE
 file_s_extattr_size_link0(VALUE path, int namespace, VALUE name)
 {
     check_namespace(namespace);
-    path = rb_str_plus(path, rb_str_new(":", 1));
-    rb_str_append(path, name);
-    HANDLE file = CreateFileW(STR2WPATH(path), 0,
+    VALUE path1 = rb_str_plus(path, rb_str_new(":", 1));
+    rb_str_append(path1, name);
+    HANDLE file = CreateFileW(STR2WPATH(path1), 0,
                               FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
                               NULL, 3,
                               FILE_FLAG_BACKUP_SEMANTICS | FILE_FLAG_OPEN_REPARSE_POINT,
                               0);
-    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError()); }
+    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError(), path); }
     return rb_ensure(extattr_size0, (VALUE)file, file_close, (VALUE)file);
 }
 
@@ -306,7 +337,7 @@ extattr_get0(VALUE args[3])
     VALUE buf = rb_str_buf_new(size);
     DWORD readsize = 0;
     if (!ReadFile(file, RSTRING_PTR(buf), size, &readsize, NULL)) {
-        raise_win32_error(GetLastError());
+        raise_win32_error(GetLastError(), Qnil);
     }
     rb_str_set_len(buf, readsize);
     return buf;
@@ -316,14 +347,14 @@ static VALUE
 extattr_get1(VALUE path, int flags, int namespace, VALUE name)
 {
     check_namespace(namespace);
-    path = rb_str_plus(path, rb_str_new(":", 1));
-    rb_str_append(path, name);
-    HANDLE file = CreateFileW(STR2WPATH(path), GENERIC_READ,
+    VALUE path1 = rb_str_plus(path, rb_str_new(":", 1));
+    rb_str_append(path1, name);
+    HANDLE file = CreateFileW(STR2WPATH(path1), GENERIC_READ,
                               FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
                               NULL, 3,
                               FILE_FLAG_BACKUP_SEMANTICS | flags,
                               0);
-    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError()); }
+    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError(), path); }
     VALUE args[] = { (VALUE)file, (VALUE)namespace, name, };
     return rb_ensure(extattr_get0, (VALUE)args, file_close, (VALUE)file);
 }
@@ -351,7 +382,7 @@ extattr_set0(VALUE args[])
     if (size > EXTATTR_MAX) { rb_raise(rb_eSystemCallError, "extattr too huge"); }
     DWORD wrotesize = size;
     if (!WriteFile(file, RSTRING_PTR(data), size, &wrotesize, NULL)) {
-        raise_win32_error(GetLastError());
+        raise_win32_error(GetLastError(), Qnil);
     }
     return Qnil;
 }
@@ -360,14 +391,14 @@ static VALUE
 file_s_extattr_set0(VALUE path, int namespace, VALUE name, VALUE data)
 {
     check_namespace(namespace);
-    path = rb_str_plus(path, rb_str_new(":", 1));
-    rb_str_append(path, name);
-    HANDLE file = CreateFileW(STR2WPATH(path), GENERIC_WRITE,
+    VALUE path1 = rb_str_plus(path, rb_str_new(":", 1));
+    rb_str_append(path1, name);
+    HANDLE file = CreateFileW(STR2WPATH(path1), GENERIC_WRITE,
                               FILE_SHARE_DELETE | FILE_SHARE_READ | FILE_SHARE_WRITE,
                               NULL, CREATE_ALWAYS,
                               FILE_FLAG_BACKUP_SEMANTICS,
                               0);
-    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError()); }
+    if (file == INVALID_HANDLE_VALUE) { raise_win32_error(GetLastError(), path); }
     VALUE args[] = { (VALUE)file, (VALUE)namespace, name, data };
     return rb_ensure(extattr_set0, (VALUE)args, file_close, (VALUE)file);
 }
@@ -382,9 +413,11 @@ static VALUE
 file_s_extattr_delete0(VALUE path, int namespace, VALUE name)
 {
     check_namespace(namespace);
-    path = rb_str_plus(path, rb_str_new(":", 1));
-    rb_str_append(path, name);
-    DeleteFileW(STR2WPATH(path));
+    VALUE path1 = rb_str_plus(path, rb_str_new(":", 1));
+    rb_str_append(path1, name);
+    if (!DeleteFileW(STR2WPATH(path1))) {
+        raise_win32_error(GetLastError(), path);
+    }
     return Qnil;
 }
 

data/ext/extconf.rb

@@ -9,10 +9,13 @@ require "mkmf"
 
 case
 when have_header("sys/extattr.h")
-when have_header("attr/xattr.h")
 when have_header("winnt.h") && have_header("ntdef.h") && have_header("psapi.h") &&
      have_header("ddk/ntifs.h") && have_header("ddk/winddk.h") &&
      have_library("ntoskrnl") && have_library("ntdll") && have_library("psapi")
+when have_header("attr/xattr.h")
+    $CPPFLAGS << " -DLINUX_XATTR_H=\\<attr/xattr.h\\>"
+when have_header("sys/xattr.h")
+    $CPPFLAGS << " -DLINUX_XATTR_H=\\<sys/xattr.h\\>"
 else
     $stderr.puts <<-EOS
 #$0: not supported target.

data/rspecs/extattr.rb

@@ -3,6 +3,7 @@
 require "tmpdir"
 basedir = File.join(Dir.tmpdir, "ruby-extattr.test-work")
 Dir.mkdir basedir unless File.directory? basedir
+filepath = File.join(basedir, "file1")
 
 require "extattr"
 
@@ -12,7 +13,7 @@ Dir.chdir basedir do
     describe "file" do
         file = nil
         before(:all) do
-            file = File.open("file1", "a")
+            file = File.open(filepath, "a")
         end
 
         it ".extattr_list" do
@@ -26,28 +27,36 @@ Dir.chdir basedir do
     end
 
     describe File do
+        before(:all) do
+            File.open(filepath, "a") {}
+        end
+
         it ".extattr_list" do
-            File.extattr_list("file1").should eq([])
+            File.extattr_list(filepath).should eq([])
         end
 
         it ".extattr_set" do
-            File.extattr_set("file1", "ext1", extdata).should nil
+            File.extattr_set(filepath, "ext1", extdata).should nil
         end
 
         it ".extattr_list ((2))" do
-            File.extattr_list("file1").should eq(["ext1"])
+            File.extattr_list(filepath).should eq(["ext1"])
         end
 
         it ".extattr_get" do
-            File.extattr_get("file1", "ext1").should eq(extdata)
+            File.extattr_get(filepath, "ext1").should eq(extdata)
         end
 
         it ".extattr_delete" do
-            File.extattr_delete("file1", "ext1").should nil
+            File.extattr_delete(filepath, "ext1").should nil
         end
 
         it ".extattr_list ((3))" do
-            File.extattr_list("file1").should eq([])
+            File.extattr_list(filepath).should eq([])
+        end
+
+        after(:all) do
+            File.unlink filepath
         end
     end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: extattr
 version: !ruby/object:Gem::Version
-  version: '0.1'
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-02 00:00:00.000000000 Z
+date: 2012-10-03 00:00:00.000000000 Z
 dependencies: []
 description: ! 'extattr is extended attribute operation library for ruby.