diff options
author | Derrell Lipman <derrell.lipman@unwireduniverse.com> | 2008-02-28 11:23:20 -0500 |
---|---|---|
committer | Karolin Seeger <ks@sernet.de> | 2008-03-04 08:57:53 +0100 |
commit | c2beacfb3fca5c50fda985862a7a78f12cc6354d (patch) | |
tree | 9e712f94bb58028ae2393da70999f7e5f89faeec /source3/libsmb/clitrans.c | |
parent | 5491030872b35453d5a13417b0b7f39032b07056 (diff) | |
download | samba-c2beacfb3fca5c50fda985862a7a78f12cc6354d.tar.gz samba-c2beacfb3fca5c50fda985862a7a78f12cc6354d.tar.bz2 samba-c2beacfb3fca5c50fda985862a7a78f12cc6354d.zip |
Initial revamp of the libsmbclient interface.
The libsmbclient interface has suffered from difficulty of improvement and
feature enrichment without causing ABI breakage. Although there were a number
of issues, the primary ones were:
(a) the user of the library would manually manipulate the context structure
members, meaning that nothing in the context structure could change other
than adding stuff at the end;
(b) there were three methods of setting options: setting bits in a flags field
within the context structure, setting explicit options variables within an
options structure in the context structure, and by calling the
smbc_option_set() function;
(c) the authentication callback did not traditionally provide enough
information to the callee which required adding an option for a callback
with a different signature, and now there are requests for even more
information at the callback, requiring yet a third signature and option to
set it (if we implement that feature).
This commit provides a reorganization of the code which fixes (a) and (b).
The context structure is now entirely opaque, and there are setter and getter
functions for manipulating it. This makes maintaining ABI consistency much,
much easier.
Additionally, the options setting/getting has been unified into a single
mechanism using smbc_option_set() and smbc_option_get().
Yet to be completed is a refactoring of the authentication callback (c).
The test programs in examples/libsmbclient have been modified (if necessary;
some applications require no changes at all) for the new API and a few have
been minimally tested.
Derrell
(cherry picked from commit d4b4bae8ded824d06ad5ab0e219f71187ee5c771)
(This used to be commit 59292c0da8880afd7bac82f6a5bba009f3827ea0)
Diffstat (limited to 'source3/libsmb/clitrans.c')
0 files changed, 0 insertions, 0 deletions