summaryrefslogtreecommitdiff
path: root/source3/smbd
diff options
context:
space:
mode:
Diffstat (limited to 'source3/smbd')
-rw-r--r--source3/smbd/service.c2
-rw-r--r--source3/smbd/vfs.c22
2 files changed, 16 insertions, 8 deletions
diff --git a/source3/smbd/service.c b/source3/smbd/service.c
index b86f3beadd..fcdd9a376b 100644
--- a/source3/smbd/service.c
+++ b/source3/smbd/service.c
@@ -485,6 +485,8 @@ connection_struct *make_connection(char *service,char *user,char *password, int
/* Loadable object file */
if (!vfs_init_custom(conn)) {
+ DEBUG(0, ("vfs_init failed\n"));
+ conn_free(conn);
return NULL;
}
#else
diff --git a/source3/smbd/vfs.c b/source3/smbd/vfs.c
index 2f984aee4f..99c8e26fa8 100644
--- a/source3/smbd/vfs.c
+++ b/source3/smbd/vfs.c
@@ -94,7 +94,8 @@ int vfs_init_default(connection_struct *conn)
#ifdef HAVE_LIBDL
BOOL vfs_init_custom(connection_struct *conn)
{
- struct vfs_ops *ops, *(*fptr)(struct vfs_options *options);
+ int vfs_version = -1;
+ struct vfs_ops *ops, *(*init_fptr)(int *);
DEBUG(3, ("Initialising custom vfs hooks from %s\n",
lp_vfsobj(SNUM(conn))));
@@ -108,22 +109,27 @@ BOOL vfs_init_custom(connection_struct *conn)
/* Get handle on vfs_init() symbol */
- fptr = (struct vfs_ops *(*)(struct vfs_options *))
- dlsym(conn->dl_handle, "vfs_init");
+ init_fptr = (struct vfs_ops *(*)(int *))dlsym(conn->dl_handle, "vfs_init");
- if (fptr == NULL) {
- DEBUG(0, ("No vfs_init() symbol found in %s\n",
+ if (init_fptr == NULL) {
+ DEBUG(0, ("No vfs_init() symbol found in %s\n",
lp_vfsobj(SNUM(conn))));
- return False;
+ return False;
}
/* Initialise vfs_ops structure */
- if ((ops = fptr(NULL)) == NULL) {
+ if ((ops = init_fptr(&vfs_version)) == NULL) {
DEBUG(0, ("vfs_init function from %s failed\n", lp_vfsobj(SNUM(conn))));
- return False;
+ return False;
}
+ if (vfs_version != SMB_VFS_INTERFACE_VERSION) {
+ DEBUG(0, ("vfs_init returned wrong interface version info (was %d, should be %d)\n",
+ vfs_version, SMB_VFS_INTERFACE_VERSION ));
+ return False;
+ }
+
/* Fill in unused operations with default (disk based) ones.
There's probably a neater way to do this then a whole bunch of
if statements. */