さとうふみやす @ OSS テクノロジです。
At Tue, 12 Aug 2008 01:05:34 +0900,
KAWASUMI Junji wrote:
> NetBSD版も同様にやってみるつもりですが、でもよくよく考えてみる
> と、これは samba本体というより、FS依存なので各ディストリビュー
> ションに投げた方がいいのかなとも思い始めました。
ちょっと調べた限りでは、ファイルシステムが birthtime に
未対応の場合に st_birthtime の値がどうなるかが BSD に依って
異なるようですね。:-(
diff --git a/source/configure.in b/source/configure.in
index 7f8a97f..8d9473d 100644
--- a/source/configure.in
+++ b/source/configure.in
@@ -5913,6 +5913,9 @@ AC_CHECK_MEMBERS([struct secmethod_table.method_attrlist], , ,
AC_CHECK_MEMBERS([struct secmethod_table.method_version], , ,
[#include <usersec.h>])
+AC_CHECK_MEMBERS([struct stat.tm_birthtime],,,
+ [#include <sys/stat.h>])
+
AC_CACHE_CHECK([for SO_PEERCRED],samba_cv_HAVE_PEERCRED,[
AC_TRY_COMPILE([#include <sys/types.h>
#include <sys/socket.h>],
diff --git a/source/lib/time.c b/source/lib/time.c
index 8e28fcc..2bf4851 100644
--- a/source/lib/time.c
+++ b/source/lib/time.c
@@ -834,7 +834,13 @@ time_t get_create_time(const SMB_STRUCT_STAT *st,BOOL fake_dirs)
if(S_ISDIR(st->st_mode) && fake_dirs) {
return (time_t)315493200L; /* 1/1/1980 */
}
-
+
+#ifdef HAVE_STRUCT_STAT_TM_BIRTHTIME
+ if (st->st_birthtime != (time_t)-1) {
+ return st->st_birthtime;
+ }
+#endif
+
ret = MIN(st->st_ctime, st->st_mtime);
ret1 = MIN(ret, st->st_atime);
http://www.nabble.com/birthtime-initialization-td18584558.html
http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/sys/stat.h?rev=HEAD&content-type=text/x-cvsweb-markup
st_birthtimensec もあるかどうか調べて、両方 0 か両方 -1 なら
無視するのがいいのかなぁ。
> そもそも struct stat の st_birthtime なんてposixにあるのか？？
> と微妙な気持ちになってきました。
ということで autoconf 対応してみました。
(*BSD 環境を持ってないので実際に動くか試してないけど :-X)
> samba側でサポートすべきかOS側のsambaビルド時のパッチに組み入れ
> るべきか、判断がつきません。
> 賢者の方々のご意見を頂きたく。
autoconf + #ifdef で Samba でサポートして欲しいですね。
--
-- Name: SATOH Fumiyasu (fumiyas @ osstech co jp)
-- Business Home: http://www.OSSTech.co.jp/
-- Personal Home: http://www.SFO.jp/blog/