diff options
author | Jeremy Allison <jra@samba.org> | 2000-12-15 21:29:06 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2000-12-15 21:29:06 +0000 |
commit | 99c2693c620cd222da5561d526aa328bec426b77 (patch) | |
tree | 5f524e8a0a8fa929953df69357195ffaf83b68af /source3/include/proto.h | |
parent | 89af6fd745a6f49668bae5b5c2d239d3671fb299 (diff) | |
download | samba-99c2693c620cd222da5561d526aa328bec426b77.tar.gz samba-99c2693c620cd222da5561d526aa328bec426b77.tar.bz2 samba-99c2693c620cd222da5561d526aa328bec426b77.zip |
Added lock backouts on fail.
When chaining together long lines of bloody "if" statements, which should
logically be separated, and one of them allocates memory, remember to
*free* it *WHETHER OR NOT THE IF STATEMENTS SUCCEEDED* !!!!
Yes I do consider this a bug in the coding style of Tridge, Rusty, Tim et al. :-).
I'm just pissed 'cos this took 4 hours to track down even with an insure error report
stating me in the face and also Ben Woodward looking over the code with me :-).
Jeremy.
(This used to be commit 506b5e34c3ba16768dbc82ba21044787de160c45)
Diffstat (limited to 'source3/include/proto.h')
-rw-r--r-- | source3/include/proto.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/source3/include/proto.h b/source3/include/proto.h index 79c742e0a8..2112187865 100644 --- a/source3/include/proto.h +++ b/source3/include/proto.h @@ -3059,9 +3059,6 @@ void init_srv_share_info2(SH_INFO_2 *sh2, char *net_name, uint32 type, char *remark, uint32 perms, uint32 max_uses, uint32 num_uses, char *path, char *passwd); -void free_srv_share_info_ctr(SRV_SHARE_INFO_CTR *ctr); -void free_srv_q_net_share_enum(SRV_Q_NET_SHARE_ENUM *q_n); -void free_srv_r_net_share_enum(SRV_R_NET_SHARE_ENUM *r_n); void init_srv_q_net_share_enum(SRV_Q_NET_SHARE_ENUM *q_n, char *srv_name, uint32 info_level, uint32 preferred_len, ENUM_HND *hnd); |