to_struct() now returns a boolean which is false if not all the elements of

the structure were present in the dictionary.
(This used to be commit b26d9d7939)

source3/python/py_conv.c

@@ -78,7 +78,7 @@ PyObject *from_struct(void *s, struct pyconv *conv)
 
 /* Convert a Python dict to a structure */
 
-void to_struct(void *s, PyObject *dict, struct pyconv *conv)
+BOOL to_struct(void *s, PyObject *dict, struct pyconv *conv)
 {
 	int i;
 
@@ -86,6 +86,9 @@ void to_struct(void *s, PyObject *dict, struct pyconv *conv)
 		PyObject *obj;
 		
 		obj = PyDict_GetItemString(dict, conv[i].name);
+
+		if (!obj)
+			return False;
 		
 		switch (conv[i].type) {
 		case PY_UNISTR: {
@@ -123,4 +126,6 @@ void to_struct(void *s, PyObject *dict, struct pyconv *conv)
 			break;
 		}
 	}
+
+	return True;
 }

source3/python/py_conv.h

@@ -30,7 +30,7 @@ struct pyconv {
 };
 
 PyObject *from_struct(void *s, struct pyconv *conv);
-void to_struct(void *s, PyObject *dict, struct pyconv *conv);
+BOOL to_struct(void *s, PyObject *dict, struct pyconv *conv);
 
 /* Another version of offsetof (-: */