X-Git-Url: http://git.samba.org/samba.git/?p=ira%2Fwip.git;a=blobdiff_plain;f=prog_guide.txt;h=3814a11a4ef531d58acd1bdf25b8399a5fca82b1;hp=8ab96fb1012d6b0fec2364391d249010659b2c52;hb=dec016822ce9439b40f3d9403d42ebc76ba2e0e1;hpb=c15862e778507287bddef7967383d4b5d22eaee9 diff --git a/prog_guide.txt b/prog_guide.txt index 8ab96fb1012..3814a11a4ef 100644 --- a/prog_guide.txt +++ b/prog_guide.txt @@ -146,6 +146,7 @@ Static data is evil as it has the following consequences: - it makes code much less likely to be recursion-safe - it leads to subtle side effects when the same code is called from multiple places + - doesn't play well with shared libraries or plugins Static data is particularly evil in library code (such as our internal smb and rpc libraries). If you can get rid of all static data in @@ -193,8 +194,9 @@ in the data and bss columns in "size" anyway (it will be included in How to use talloc ----------------- -Please see the separate document, talloc_guide.txt in this -directory. You _must_ read this if you want to program in Samba4. +Please see the separate document, source/lib/talloc/talloc_guide.txt +You _must_ read this if you want to program in Samba4. + Interface Structures -------------------- @@ -206,7 +208,7 @@ an idea of what I am talking about. In Samba3 many of the core wire structures in the SMB protocol were never explicitly defined in Samba. Instead, our parse and generation functions just worked directly with wire buffers. The biggest problem -with this is that is tied our parse code with out "business logic" +with this is that is tied our parse code with our "business logic" much too closely, which meant the code got extremely confusing to read. @@ -231,7 +233,7 @@ msrpc code from Samba3", and while to some extent this is true there are extremely important differences in the approach that are worth pointing out. -In the Samba3 msrpc code we used explicit parse strucrures for all +In the Samba3 msrpc code we used explicit parse structures for all msrpc functions. The problem is that we didn't just put all of the real variables in these structures, we also put in all the artifacts as well. A good example is the security descriptor strucrure that @@ -262,7 +264,7 @@ parser where to find the following four variables, but they should *NOT* be in the interface structure. In Samba3 there were unwritten rules about which variables in a -strucrure a high level caller has to fill in and which ones are filled +structure a high level caller has to fill in and which ones are filled in by the marshalling code. In Samba4 those rules are gone, because the redundent artifact variables are gone. The high level caller just sets up the real variables and the marshalling code worries about @@ -394,7 +396,7 @@ function, so smbd has a _send() function and the parse function for each SMB. As an example go and have a look at reply_getatr_send() and -reply_getatr() in smbd/reply.c. Read them? Good. +reply_getatr() in smb_server/reply.c. Read them? Good. Notice that reply_getatr() sets up the req->async structure to contain the send function. Thats how the backend gets to do an async reply, it @@ -539,30 +541,33 @@ will be auto-determined. other recognised flags are: - sign : enable ntlmssp signing - seal : enable ntlmssp sealing - connect : enable rpc connect level auth (auth, but no sign or seal) - validate: enable the NDR validator - print: enable debugging of the packets - bigendian: use bigendian RPC - padcheck: check reply data for non-zero pad bytes + sign : enable ntlmssp signing + seal : enable ntlmssp sealing + spnego : use SPNEGO instead of NTLMSSP authentication + krb5 : use KRB5 instead of NTLMSSP authentication + connect : enable rpc connect level auth (auth, but no sign or seal) + validate : enable the NDR validator + print : enable debugging of the packets + bigendian : use bigendian RPC + padcheck : check reply data for non-zero pad bytes -For example, these all connect to the samr pipe: +Here are some examples: ncacn_np:myserver ncacn_np:myserver[samr] ncacn_np:myserver[\pipe\samr] ncacn_np:myserver[/pipe/samr] ncacn_np:myserver[samr,sign,print] + ncacn_np:myserver[sign,spnego] ncacn_np:myserver[\pipe\samr,sign,seal,bigendian] ncacn_np:myserver[/pipe/samr,seal,validate] ncacn_np: ncacn_np:[/pipe/samr] - ncacn_ip_tcp:myserver ncacn_ip_tcp:myserver[1024] - ncacn_ip_tcp:myserver[1024,sign,seal] + ncacn_ip_tcp:myserver[sign,seal] + ncacn_ip_tcp:myserver[spnego,seal] IDEA: Maybe extend UNC names like this? @@ -764,3 +769,21 @@ BUGS: trans2 and other calls handle servers that don't have the setattre call in torture add max file coponent length test and max path len test + check for alloc failure in all core reply.c and trans2.c code where + allocation size depends on client parameter + +case-insenstive idea: + all filenames on disk lowercase + real case in extended attribute + keep cache of what dirs are all lowercase + when searching for name, don't search if dir is definately all lowercase + when creating file, use dnotify to tell if someone else creates at + same time + +solve del *.* idea: + make mangle cache dynamic size + fill during a dir scan + setup a timer + destroy cache after 30 sec + destroy if a 2nd dir scan happens on same dir +