[PATCH] Two vfs_fruit cleanup patches

Volker Lendecke Volker.Lendecke at SerNet.DE
Wed Dec 17 06:52:33 MST 2014


Hi!

Review&push appreciated!

Thanks,

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de
-------------- next part --------------
From 86569e9487c9c956acc57948f8f860b464d877e0 Mon Sep 17 00:00:00 2001
From: Volker Lendecke <vl at samba.org>
Date: Tue, 16 Dec 2014 09:38:21 +0100
Subject: [PATCH 1/2] vfs_fruit: Avoid double initialization

Signed-off-by: Volker Lendecke <vl at samba.org>
---
 source3/modules/vfs_fruit.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/source3/modules/vfs_fruit.c b/source3/modules/vfs_fruit.c
index a8bf7b4..d11e1bd 100644
--- a/source3/modules/vfs_fruit.c
+++ b/source3/modules/vfs_fruit.c
@@ -3389,7 +3389,6 @@ static struct vfs_fn_pointers vfs_fruit_fns = {
 	.fstat_fn = fruit_fstat,
 	.streaminfo_fn = fruit_streaminfo,
 	.ntimes_fn = fruit_ntimes,
-	.unlink_fn = fruit_unlink,
 	.ftruncate_fn = fruit_ftruncate,
 	.fallocate_fn = fruit_fallocate,
 	.create_file_fn = fruit_create_file,
-- 
1.9.1


From 44cd1510ab65257b7665ee1eab52e7024ce7ca05 Mon Sep 17 00:00:00 2001
From: Volker Lendecke <vl at samba.org>
Date: Tue, 16 Dec 2014 09:38:54 +0100
Subject: [PATCH 2/2] vfs_fruit: Avoid double ()

---
 source3/modules/vfs_fruit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source3/modules/vfs_fruit.c b/source3/modules/vfs_fruit.c
index d11e1bd..ecaddfb 100644
--- a/source3/modules/vfs_fruit.c
+++ b/source3/modules/vfs_fruit.c
@@ -526,7 +526,7 @@ static bool ad_pack(struct adouble *ad)
 	offset += ADEDLEN_NENTRIES;
 
 	for (eid = 0, nent = 0; eid < ADEID_MAX; eid++) {
-		if ((ad->ad_eid[eid].ade_off == 0)) {
+		if (ad->ad_eid[eid].ade_off == 0) {
 			/*
 			 * ade_off is also used as indicator whether a
 			 * specific entry is used or not
-- 
1.9.1



More information about the samba-technical mailing list