rewrote enumprinterdata. still a bug in it but reproducing it hard and
authorJean-François Micouleau <jfm@samba.org>
Sat, 26 Feb 2000 22:22:24 +0000 (22:22 +0000)
committerJean-François Micouleau <jfm@samba.org>
Sat, 26 Feb 2000 22:22:24 +0000 (22:22 +0000)
borring.

I need a client test program urgently!!!

rewrote setprinter, doesn't coredump anymore, and no memleak.

J.F.
(This used to be commit b76ae1f92f4f12b38c4245456cdd2db970724077)

source3/include/proto.h
source3/include/rpc_spoolss.h
source3/rpc_parse/parse_spoolss.c
source3/rpc_server/srv_spoolss.c
source3/rpc_server/srv_spoolss_nt.c

index 830e7bc5b1fd3735cfe6b2a4cce0769ebc13604a..53daacc098b18b99a5851788dfe357b9f5abe663 100644 (file)
@@ -186,10 +186,208 @@ void CatchChildLeaveStatus(void);
 
 int vslprintf(char *str, int n, char *format, va_list ap);
 
+/*The following definitions come from  libsmb/clientgen.c  */
+
+int cli_set_port(struct cli_state *cli, int port);
+char *cli_errstr(struct cli_state *cli);
+BOOL cli_api_pipe(struct cli_state *cli, char *pipe_name, int pipe_name_len,
+                  uint16 *setup, uint32 setup_count, uint32 max_setup_count,
+                  char *params, uint32 param_count, uint32 max_param_count,
+                  char *data, uint32 data_count, uint32 max_data_count,
+                  char **rparam, uint32 *rparam_count,
+                  char **rdata, uint32 *rdata_count);
+BOOL cli_api(struct cli_state *cli,
+            char *param, int prcnt, int mprcnt,
+            char *data, int drcnt, int mdrcnt,
+            char **rparam, int *rprcnt,
+            char **rdata, int *rdrcnt);
+BOOL cli_NetWkstaUserLogon(struct cli_state *cli,char *user, char *workstation);
+int cli_RNetShareEnum(struct cli_state *cli, void (*fn)(const char *, uint32, const char *));
+BOOL cli_NetServerEnum(struct cli_state *cli, char *workgroup, uint32 stype,
+                      void (*fn)(const char *, uint32, const char *));
+BOOL cli_session_setup(struct cli_state *cli, 
+                      char *user, 
+                      char *pass, int passlen,
+                      char *ntpass, int ntpasslen,
+                      char *workgroup);
+BOOL cli_ulogoff(struct cli_state *cli);
+BOOL cli_send_tconX(struct cli_state *cli, 
+                   char *share, char *dev, char *pass, int passlen);
+BOOL cli_tdis(struct cli_state *cli);
+BOOL cli_rename(struct cli_state *cli, char *fname_src, char *fname_dst);
+BOOL cli_unlink(struct cli_state *cli, char *fname);
+BOOL cli_mkdir(struct cli_state *cli, char *dname);
+BOOL cli_rmdir(struct cli_state *cli, char *dname);
+int cli_nt_create(struct cli_state *cli, char *fname);
+int cli_open(struct cli_state *cli, char *fname, int flags, int share_mode);
+BOOL cli_close(struct cli_state *cli, int fnum);
+BOOL cli_lock(struct cli_state *cli, int fnum, 
+             uint32 offset, uint32 len, int timeout, enum brl_type lock_type);
+BOOL cli_unlock(struct cli_state *cli, int fnum, uint32 offset, uint32 len);
+size_t cli_read(struct cli_state *cli, int fnum, char *buf, off_t offset, size_t size);
+ssize_t cli_write(struct cli_state *cli,
+                 int fnum, uint16 write_mode,
+                 char *buf, off_t offset, size_t size);
+ssize_t cli_smbwrite(struct cli_state *cli,
+                    int fnum, char *buf, off_t offset, size_t size1);
+BOOL cli_getattrE(struct cli_state *cli, int fd, 
+                 uint16 *attr, size_t *size, 
+                 time_t *c_time, time_t *a_time, time_t *m_time);
+BOOL cli_getatr(struct cli_state *cli, char *fname, 
+               uint16 *attr, size_t *size, time_t *t);
+BOOL cli_setatr(struct cli_state *cli, char *fname, uint16 attr, time_t t);
+BOOL cli_qpathinfo(struct cli_state *cli, const char *fname, 
+                  time_t *c_time, time_t *a_time, time_t *m_time, 
+                  size_t *size, uint16 *mode);
+BOOL cli_qpathinfo2(struct cli_state *cli, const char *fname, 
+                   time_t *c_time, time_t *a_time, time_t *m_time, 
+                   time_t *w_time, size_t *size, uint16 *mode,
+                   SMB_INO_T *ino);
+BOOL cli_qfileinfo(struct cli_state *cli, int fnum, 
+                  uint16 *mode, size_t *size,
+                  time_t *c_time, time_t *a_time, time_t *m_time, 
+                  time_t *w_time, SMB_INO_T *ino);
+int cli_list(struct cli_state *cli,const char *Mask,uint16 attribute, 
+            void (*fn)(file_info *, const char *));
+BOOL cli_oem_change_password(struct cli_state *cli, const char *user, const char *new_password,
+                             const char *old_password);
+BOOL cli_negprot(struct cli_state *cli);
+BOOL cli_session_request(struct cli_state *cli,
+                        struct nmb_name *calling, struct nmb_name *called);
+BOOL cli_connect(struct cli_state *cli, const char *host, struct in_addr *ip);
+struct cli_state *cli_initialise(struct cli_state *cli);
+void cli_shutdown(struct cli_state *cli);
+int cli_error(struct cli_state *cli, uint8 *eclass, uint32 *num, uint32 *nt_rpc_error);
+void cli_sockopt(struct cli_state *cli, char *options);
+uint16 cli_setpid(struct cli_state *cli, uint16 pid);
+BOOL cli_reestablish_connection(struct cli_state *cli);
+BOOL cli_establish_connection(struct cli_state *cli, 
+                               char *dest_host, struct in_addr *dest_ip,
+                               struct nmb_name *calling, struct nmb_name *called,
+                               char *service, char *service_type,
+                               BOOL do_shutdown, BOOL do_tcon);
+int cli_printjob_del(struct cli_state *cli, int job);
+int cli_print_queue(struct cli_state *cli, 
+                   void (*fn)(struct print_job_info *));
+BOOL cli_chkpath(struct cli_state *cli, char *path);
+BOOL cli_message_start(struct cli_state *cli, char *host, char *username, 
+                             int *grp);
+BOOL cli_message_text(struct cli_state *cli, char *msg, int len, int grp);
+BOOL cli_message_end(struct cli_state *cli, int grp);
+BOOL cli_dskattr(struct cli_state *cli, int *bsize, int *total, int *avail);
+BOOL attempt_netbios_session_request(struct cli_state *cli, char *srchost, char *desthost,
+                                     struct in_addr *pdest_ip);
+
+/*The following definitions come from  libsmb/credentials.c  */
+
+char *credstr(uchar *cred);
+void cred_session_key(DOM_CHAL *clnt_chal, DOM_CHAL *srv_chal, char *pass, 
+                     uchar session_key[8]);
+void cred_create(uchar session_key[8], DOM_CHAL *stor_cred, UTIME timestamp, 
+                DOM_CHAL *cred);
+int cred_assert(DOM_CHAL *cred, uchar session_key[8], DOM_CHAL *stored_cred,
+               UTIME timestamp);
+BOOL clnt_deal_with_creds(uchar sess_key[8],
+                         DOM_CRED *sto_clnt_cred, DOM_CRED *rcv_srv_cred);
+BOOL deal_with_creds(uchar sess_key[8],
+                    DOM_CRED *sto_clnt_cred, 
+                    DOM_CRED *rcv_clnt_cred, DOM_CRED *rtn_srv_cred);
+
+/*The following definitions come from  libsmb/namequery.c  */
+
+BOOL name_status(int fd,char *name,int name_type,BOOL recurse,
+                struct in_addr to_ip,char *master,char *rname);
+struct in_addr *name_query(int fd,const char *name,int name_type, 
+                          BOOL bcast,BOOL recurse,
+                          struct in_addr to_ip, int *count);
+FILE *startlmhosts(char *fname);
+BOOL getlmhostsent( FILE *fp, pstring name, int *name_type, struct in_addr *ipaddr);
+void endlmhosts(FILE *fp);
+BOOL resolve_name(const char *name, struct in_addr *return_ip, int name_type);
+BOOL find_master_ip(char *group, struct in_addr *master_ip);
+BOOL lookup_pdc_name(const char *srcname, const char *domain, struct in_addr *pdc_ip, char *ret_name);
+BOOL get_dc_list(char *group, struct in_addr **ip_list, int *count);
+
+/*The following definitions come from  libsmb/nmblib.c  */
+
+void debug_nmb_packet(struct packet_struct *p);
+char *nmb_namestr(struct nmb_name *n);
+struct packet_struct *copy_packet(struct packet_struct *packet);
+void free_packet(struct packet_struct *packet);
+struct packet_struct *parse_packet(char *buf,int length,
+                                  enum packet_type packet_type);
+struct packet_struct *read_packet(int fd,enum packet_type packet_type);
+void make_nmb_name( struct nmb_name *n, const char *name, int type);
+BOOL nmb_name_equal(struct nmb_name *n1, struct nmb_name *n2);
+int build_packet(char *buf, struct packet_struct *p);
+BOOL send_packet(struct packet_struct *p);
+struct packet_struct *receive_packet(int fd,enum packet_type type,int t);
+struct packet_struct *receive_nmb_packet(int fd, int t, int trn_id);
+struct packet_struct *receive_dgram_packet(int fd, int t, char *mailslot_name);
+BOOL match_mailslot_name(struct packet_struct *p, char *mailslot_name);
+void sort_query_replies(char *data, int n, struct in_addr ip);
+
+/*The following definitions come from  libsmb/nterr.c  */
+
+char *get_nt_error_msg(uint32 nt_code);
+
+/*The following definitions come from  libsmb/passchange.c  */
+
+BOOL remote_password_change(const char *remote_machine, const char *user_name, 
+                           const char *old_passwd, const char *new_passwd,
+                           char *err_str, size_t err_str_len);
+
+/*The following definitions come from  libsmb/pwd_cache.c  */
+
+void pwd_init(struct pwd_info *pwd);
+void pwd_obfuscate_key(struct pwd_info *pwd, uint32 int_key, char *str_key);
+BOOL pwd_compare(struct pwd_info *pwd1, struct pwd_info *pwd2);
+void pwd_read(struct pwd_info *pwd, char *passwd_report, BOOL do_encrypt);
+void pwd_set_nullpwd(struct pwd_info *pwd);
+void pwd_set_cleartext(struct pwd_info *pwd, char *clr);
+void pwd_get_cleartext(struct pwd_info *pwd, char *clr);
+void pwd_set_lm_nt_16(struct pwd_info *pwd, uchar lm_pwd[16], uchar nt_pwd[16]);
+void pwd_get_lm_nt_16(struct pwd_info *pwd, uchar lm_pwd[16], uchar nt_pwd[16]);
+void pwd_make_lm_nt_16(struct pwd_info *pwd, char *clr);
+void pwd_make_lm_nt_owf(struct pwd_info *pwd, uchar cryptkey[8]);
+void pwd_get_lm_nt_owf(struct pwd_info *pwd, uchar lm_owf[24], uchar nt_owf[24]);
+
 /*The following definitions come from  lib/smbrun.c  */
 
 int smbrun(char *cmd,char *outfile,BOOL shared);
 
+/*The following definitions come from  libsmb/smbdes.c  */
+
+void E_P16(unsigned char *p14,unsigned char *p16);
+void E_P24(unsigned char *p21, unsigned char *c8, unsigned char *p24);
+void D_P16(unsigned char *p14, unsigned char *in, unsigned char *out);
+void E_old_pw_hash( unsigned char *p14, unsigned char *in, unsigned char *out);
+void cred_hash1(unsigned char *out,unsigned char *in,unsigned char *key);
+void cred_hash2(unsigned char *out,unsigned char *in,unsigned char *key);
+void cred_hash3(unsigned char *out,unsigned char *in,unsigned char *key, int forw);
+void SamOEMhash( unsigned char *data, unsigned char *key, int val);
+
+/*The following definitions come from  libsmb/smbencrypt.c  */
+
+void SMBencrypt(uchar *passwd, uchar *c8, uchar *p24);
+void E_md4hash(uchar *passwd, uchar *p16);
+void nt_lm_owf_gen(char *pwd, uchar nt_p16[16], uchar p16[16]);
+void SMBOWFencrypt(uchar passwd[16], uchar *c8, uchar p24[24]);
+void NTLMSSPOWFencrypt(uchar passwd[8], uchar *ntlmchalresp, uchar p24[24]);
+void SMBNTencrypt(uchar *passwd, uchar *c8, uchar *p24);
+BOOL make_oem_passwd_hash(char data[516], const char *passwd, uchar old_pw_hash[16], BOOL unicode);
+
+/*The following definitions come from  libsmb/smberr.c  */
+
+char *smb_errstr(char *inbuf);
+
+/*The following definitions come from  libsmb/unexpected.c  */
+
+void unexpected_packet(struct packet_struct *p);
+void clear_unexpected(time_t t);
+struct packet_struct *receive_unexpected(enum packet_type packet_type, int id, 
+                                        char *mailslot_name);
+
 /*The following definitions come from  lib/snprintf.c  */
 
 
@@ -274,6 +472,23 @@ BOOL map_username(char *user);
 struct passwd *Get_Pwnam(char *user,BOOL allow_change);
 BOOL user_in_list(char *user,char *list);
 
+/*The following definitions come from  lib/util_array.c  */
+
+void free_void_array(uint32 num_entries, void **entries,
+               void(free_item)(void*));
+void* add_copy_to_array(uint32 *len, void ***array, const void *item,
+       void*(item_dup)(const void*), BOOL alloc_anyway);
+void* add_item_to_array(uint32 *len, void ***array, void *item);
+void free_use_info_array(uint32 num_entries, struct use_info **entries);
+struct use_info* add_use_info_to_array(uint32 *len, struct use_info ***array,
+                               const struct use_info *name);
+void free_char_array(uint32 num_entries, char **entries);
+char* add_chars_to_array(uint32 *len, char ***array, const char *name);
+void free_uint32_array(uint32 num_entries, uint32 **entries);
+uint32* add_uint32s_to_array(uint32 *len, uint32 ***array, const uint32 *name);
+void free_sid_array(uint32 num_entries, DOM_SID **entries);
+DOM_SID* add_sid_to_array(uint32 *len, DOM_SID ***array, const DOM_SID *sid);
+
 /*The following definitions come from  lib/util.c  */
 
 char *tmpdir(void);
@@ -351,23 +566,6 @@ char *myhostname(void);
 char *lock_path(char *name);
 char *parent_dirname(const char *path);
 
-/*The following definitions come from  lib/util_array.c  */
-
-void free_void_array(uint32 num_entries, void **entries,
-               void(free_item)(void*));
-void* add_copy_to_array(uint32 *len, void ***array, const void *item,
-       void*(item_dup)(const void*), BOOL alloc_anyway);
-void* add_item_to_array(uint32 *len, void ***array, void *item);
-void free_use_info_array(uint32 num_entries, struct use_info **entries);
-struct use_info* add_use_info_to_array(uint32 *len, struct use_info ***array,
-                               const struct use_info *name);
-void free_char_array(uint32 num_entries, char **entries);
-char* add_chars_to_array(uint32 *len, char ***array, const char *name);
-void free_uint32_array(uint32 num_entries, uint32 **entries);
-uint32* add_uint32s_to_array(uint32 *len, uint32 ***array, const uint32 *name);
-void free_sid_array(uint32 num_entries, DOM_SID **entries);
-DOM_SID* add_sid_to_array(uint32 *len, DOM_SID ***array, const DOM_SID *sid);
-
 /*The following definitions come from  lib/util_file.c  */
 
 BOOL do_file_lock(int fd, int waitsecs, int type);
@@ -554,206 +752,8 @@ void fstring_sub_w(smb_ucs2_t *s,const smb_ucs2_t *pattern,const smb_ucs2_t *ins
 void pstring_sub_w(smb_ucs2_t *s,const smb_ucs2_t *pattern,smb_ucs2_t *insert);
 void all_string_sub_w(smb_ucs2_t *s,const smb_ucs2_t *pattern,const smb_ucs2_t *insert, size_t len);
 void split_at_last_component_w(smb_ucs2_t *path, smb_ucs2_t *front, smb_ucs2_t sep, smb_ucs2_t *back);
-smb_ucs2_t *octal_string_w(int i);
-smb_ucs2_t *string_truncate_w(smb_ucs2_t *s, size_t length);
-
-/*The following definitions come from  libsmb/clientgen.c  */
-
-int cli_set_port(struct cli_state *cli, int port);
-char *cli_errstr(struct cli_state *cli);
-BOOL cli_api_pipe(struct cli_state *cli, char *pipe_name, int pipe_name_len,
-                  uint16 *setup, uint32 setup_count, uint32 max_setup_count,
-                  char *params, uint32 param_count, uint32 max_param_count,
-                  char *data, uint32 data_count, uint32 max_data_count,
-                  char **rparam, uint32 *rparam_count,
-                  char **rdata, uint32 *rdata_count);
-BOOL cli_api(struct cli_state *cli,
-            char *param, int prcnt, int mprcnt,
-            char *data, int drcnt, int mdrcnt,
-            char **rparam, int *rprcnt,
-            char **rdata, int *rdrcnt);
-BOOL cli_NetWkstaUserLogon(struct cli_state *cli,char *user, char *workstation);
-int cli_RNetShareEnum(struct cli_state *cli, void (*fn)(const char *, uint32, const char *));
-BOOL cli_NetServerEnum(struct cli_state *cli, char *workgroup, uint32 stype,
-                      void (*fn)(const char *, uint32, const char *));
-BOOL cli_session_setup(struct cli_state *cli, 
-                      char *user, 
-                      char *pass, int passlen,
-                      char *ntpass, int ntpasslen,
-                      char *workgroup);
-BOOL cli_ulogoff(struct cli_state *cli);
-BOOL cli_send_tconX(struct cli_state *cli, 
-                   char *share, char *dev, char *pass, int passlen);
-BOOL cli_tdis(struct cli_state *cli);
-BOOL cli_rename(struct cli_state *cli, char *fname_src, char *fname_dst);
-BOOL cli_unlink(struct cli_state *cli, char *fname);
-BOOL cli_mkdir(struct cli_state *cli, char *dname);
-BOOL cli_rmdir(struct cli_state *cli, char *dname);
-int cli_nt_create(struct cli_state *cli, char *fname);
-int cli_open(struct cli_state *cli, char *fname, int flags, int share_mode);
-BOOL cli_close(struct cli_state *cli, int fnum);
-BOOL cli_lock(struct cli_state *cli, int fnum, 
-             uint32 offset, uint32 len, int timeout, enum brl_type lock_type);
-BOOL cli_unlock(struct cli_state *cli, int fnum, uint32 offset, uint32 len);
-size_t cli_read(struct cli_state *cli, int fnum, char *buf, off_t offset, size_t size);
-ssize_t cli_write(struct cli_state *cli,
-                 int fnum, uint16 write_mode,
-                 char *buf, off_t offset, size_t size);
-ssize_t cli_smbwrite(struct cli_state *cli,
-                    int fnum, char *buf, off_t offset, size_t size1);
-BOOL cli_getattrE(struct cli_state *cli, int fd, 
-                 uint16 *attr, size_t *size, 
-                 time_t *c_time, time_t *a_time, time_t *m_time);
-BOOL cli_getatr(struct cli_state *cli, char *fname, 
-               uint16 *attr, size_t *size, time_t *t);
-BOOL cli_setatr(struct cli_state *cli, char *fname, uint16 attr, time_t t);
-BOOL cli_qpathinfo(struct cli_state *cli, const char *fname, 
-                  time_t *c_time, time_t *a_time, time_t *m_time, 
-                  size_t *size, uint16 *mode);
-BOOL cli_qpathinfo2(struct cli_state *cli, const char *fname, 
-                   time_t *c_time, time_t *a_time, time_t *m_time, 
-                   time_t *w_time, size_t *size, uint16 *mode,
-                   SMB_INO_T *ino);
-BOOL cli_qfileinfo(struct cli_state *cli, int fnum, 
-                  uint16 *mode, size_t *size,
-                  time_t *c_time, time_t *a_time, time_t *m_time, 
-                  time_t *w_time, SMB_INO_T *ino);
-int cli_list(struct cli_state *cli,const char *Mask,uint16 attribute, 
-            void (*fn)(file_info *, const char *));
-BOOL cli_oem_change_password(struct cli_state *cli, const char *user, const char *new_password,
-                             const char *old_password);
-BOOL cli_negprot(struct cli_state *cli);
-BOOL cli_session_request(struct cli_state *cli,
-                        struct nmb_name *calling, struct nmb_name *called);
-BOOL cli_connect(struct cli_state *cli, const char *host, struct in_addr *ip);
-struct cli_state *cli_initialise(struct cli_state *cli);
-void cli_shutdown(struct cli_state *cli);
-int cli_error(struct cli_state *cli, uint8 *eclass, uint32 *num, uint32 *nt_rpc_error);
-void cli_sockopt(struct cli_state *cli, char *options);
-uint16 cli_setpid(struct cli_state *cli, uint16 pid);
-BOOL cli_reestablish_connection(struct cli_state *cli);
-BOOL cli_establish_connection(struct cli_state *cli, 
-                               char *dest_host, struct in_addr *dest_ip,
-                               struct nmb_name *calling, struct nmb_name *called,
-                               char *service, char *service_type,
-                               BOOL do_shutdown, BOOL do_tcon);
-int cli_printjob_del(struct cli_state *cli, int job);
-int cli_print_queue(struct cli_state *cli, 
-                   void (*fn)(struct print_job_info *));
-BOOL cli_chkpath(struct cli_state *cli, char *path);
-BOOL cli_message_start(struct cli_state *cli, char *host, char *username, 
-                             int *grp);
-BOOL cli_message_text(struct cli_state *cli, char *msg, int len, int grp);
-BOOL cli_message_end(struct cli_state *cli, int grp);
-BOOL cli_dskattr(struct cli_state *cli, int *bsize, int *total, int *avail);
-BOOL attempt_netbios_session_request(struct cli_state *cli, char *srchost, char *desthost,
-                                     struct in_addr *pdest_ip);
-
-/*The following definitions come from  libsmb/credentials.c  */
-
-char *credstr(uchar *cred);
-void cred_session_key(DOM_CHAL *clnt_chal, DOM_CHAL *srv_chal, char *pass, 
-                     uchar session_key[8]);
-void cred_create(uchar session_key[8], DOM_CHAL *stor_cred, UTIME timestamp, 
-                DOM_CHAL *cred);
-int cred_assert(DOM_CHAL *cred, uchar session_key[8], DOM_CHAL *stored_cred,
-               UTIME timestamp);
-BOOL clnt_deal_with_creds(uchar sess_key[8],
-                         DOM_CRED *sto_clnt_cred, DOM_CRED *rcv_srv_cred);
-BOOL deal_with_creds(uchar sess_key[8],
-                    DOM_CRED *sto_clnt_cred, 
-                    DOM_CRED *rcv_clnt_cred, DOM_CRED *rtn_srv_cred);
-
-/*The following definitions come from  libsmb/namequery.c  */
-
-BOOL name_status(int fd,char *name,int name_type,BOOL recurse,
-                struct in_addr to_ip,char *master,char *rname);
-struct in_addr *name_query(int fd,const char *name,int name_type, 
-                          BOOL bcast,BOOL recurse,
-                          struct in_addr to_ip, int *count);
-FILE *startlmhosts(char *fname);
-BOOL getlmhostsent( FILE *fp, pstring name, int *name_type, struct in_addr *ipaddr);
-void endlmhosts(FILE *fp);
-BOOL resolve_name(const char *name, struct in_addr *return_ip, int name_type);
-BOOL find_master_ip(char *group, struct in_addr *master_ip);
-BOOL lookup_pdc_name(const char *srcname, const char *domain, struct in_addr *pdc_ip, char *ret_name);
-BOOL get_dc_list(char *group, struct in_addr **ip_list, int *count);
-
-/*The following definitions come from  libsmb/nmblib.c  */
-
-void debug_nmb_packet(struct packet_struct *p);
-char *nmb_namestr(struct nmb_name *n);
-struct packet_struct *copy_packet(struct packet_struct *packet);
-void free_packet(struct packet_struct *packet);
-struct packet_struct *parse_packet(char *buf,int length,
-                                  enum packet_type packet_type);
-struct packet_struct *read_packet(int fd,enum packet_type packet_type);
-void make_nmb_name( struct nmb_name *n, const char *name, int type);
-BOOL nmb_name_equal(struct nmb_name *n1, struct nmb_name *n2);
-int build_packet(char *buf, struct packet_struct *p);
-BOOL send_packet(struct packet_struct *p);
-struct packet_struct *receive_packet(int fd,enum packet_type type,int t);
-struct packet_struct *receive_nmb_packet(int fd, int t, int trn_id);
-struct packet_struct *receive_dgram_packet(int fd, int t, char *mailslot_name);
-BOOL match_mailslot_name(struct packet_struct *p, char *mailslot_name);
-void sort_query_replies(char *data, int n, struct in_addr ip);
-
-/*The following definitions come from  libsmb/nterr.c  */
-
-char *get_nt_error_msg(uint32 nt_code);
-
-/*The following definitions come from  libsmb/passchange.c  */
-
-BOOL remote_password_change(const char *remote_machine, const char *user_name, 
-                           const char *old_passwd, const char *new_passwd,
-                           char *err_str, size_t err_str_len);
-
-/*The following definitions come from  libsmb/pwd_cache.c  */
-
-void pwd_init(struct pwd_info *pwd);
-void pwd_obfuscate_key(struct pwd_info *pwd, uint32 int_key, char *str_key);
-BOOL pwd_compare(struct pwd_info *pwd1, struct pwd_info *pwd2);
-void pwd_read(struct pwd_info *pwd, char *passwd_report, BOOL do_encrypt);
-void pwd_set_nullpwd(struct pwd_info *pwd);
-void pwd_set_cleartext(struct pwd_info *pwd, char *clr);
-void pwd_get_cleartext(struct pwd_info *pwd, char *clr);
-void pwd_set_lm_nt_16(struct pwd_info *pwd, uchar lm_pwd[16], uchar nt_pwd[16]);
-void pwd_get_lm_nt_16(struct pwd_info *pwd, uchar lm_pwd[16], uchar nt_pwd[16]);
-void pwd_make_lm_nt_16(struct pwd_info *pwd, char *clr);
-void pwd_make_lm_nt_owf(struct pwd_info *pwd, uchar cryptkey[8]);
-void pwd_get_lm_nt_owf(struct pwd_info *pwd, uchar lm_owf[24], uchar nt_owf[24]);
-
-/*The following definitions come from  libsmb/smbdes.c  */
-
-void E_P16(unsigned char *p14,unsigned char *p16);
-void E_P24(unsigned char *p21, unsigned char *c8, unsigned char *p24);
-void D_P16(unsigned char *p14, unsigned char *in, unsigned char *out);
-void E_old_pw_hash( unsigned char *p14, unsigned char *in, unsigned char *out);
-void cred_hash1(unsigned char *out,unsigned char *in,unsigned char *key);
-void cred_hash2(unsigned char *out,unsigned char *in,unsigned char *key);
-void cred_hash3(unsigned char *out,unsigned char *in,unsigned char *key, int forw);
-void SamOEMhash( unsigned char *data, unsigned char *key, int val);
-
-/*The following definitions come from  libsmb/smbencrypt.c  */
-
-void SMBencrypt(uchar *passwd, uchar *c8, uchar *p24);
-void E_md4hash(uchar *passwd, uchar *p16);
-void nt_lm_owf_gen(char *pwd, uchar nt_p16[16], uchar p16[16]);
-void SMBOWFencrypt(uchar passwd[16], uchar *c8, uchar p24[24]);
-void NTLMSSPOWFencrypt(uchar passwd[8], uchar *ntlmchalresp, uchar p24[24]);
-void SMBNTencrypt(uchar *passwd, uchar *c8, uchar *p24);
-BOOL make_oem_passwd_hash(char data[516], const char *passwd, uchar old_pw_hash[16], BOOL unicode);
-
-/*The following definitions come from  libsmb/smberr.c  */
-
-char *smb_errstr(char *inbuf);
-
-/*The following definitions come from  libsmb/unexpected.c  */
-
-void unexpected_packet(struct packet_struct *p);
-void clear_unexpected(time_t t);
-struct packet_struct *receive_unexpected(enum packet_type packet_type, int id, 
-                                        char *mailslot_name);
+smb_ucs2_t *octal_string_w(int i);
+smb_ucs2_t *string_truncate_w(smb_ucs2_t *s, size_t length);
 
 /*The following definitions come from  locking/brlock.c  */
 
@@ -812,10 +812,6 @@ BOOL queue_dns_query(struct packet_struct *p,struct nmb_name *question,
                     struct name_record **n);
 void kill_async_dns_child(void);
 
-/*The following definitions come from  nmbd/nmbd.c  */
-
-BOOL reload_services(BOOL test);
-
 /*The following definitions come from  nmbd/nmbd_become_dmb.c  */
 
 void add_domain_names(time_t t);
@@ -846,6 +842,10 @@ void announce_and_sync_with_domain_master_browser( struct subnet_record *subrec,
 void collect_all_workgroup_names_from_wins_server(time_t t);
 void sync_all_dmbs(time_t t);
 
+/*The following definitions come from  nmbd/nmbd.c  */
+
+BOOL reload_services(BOOL test);
+
 /*The following definitions come from  nmbd/nmbd_elections.c  */
 
 void check_master_browser_exists(time_t t);
@@ -1520,11 +1520,6 @@ void pcap_printer_fn(void (*fn)(char *, char *));
 void cups_printer_fn(void (*fn)(char *, char *));
 int cups_printername_ok(char *name);
 
-/*The following definitions come from  printing/print_svid.c  */
-
-void sysv_printer_fn(void (*fn)(char *, char *));
-int sysv_printername_ok(char *name);
-
 /*The following definitions come from  printing/printing.c  */
 
 void lpq_reset(int snum);
@@ -1539,6 +1534,11 @@ void printjob_decode(int jobid, int *snum, int *job);
 void status_printqueue(connection_struct *conn,int snum,int status);
 void load_printers(void);
 
+/*The following definitions come from  printing/print_svid.c  */
+
+void sysv_printer_fn(void (*fn)(char *, char *));
+int sysv_printername_ok(char *name);
+
 /*The following definitions come from  profile/profile.c  */
 
 BOOL profile_setup(BOOL rdonly);
@@ -1704,6 +1704,109 @@ BOOL do_wks_query_info(struct cli_state *cli,
                        char *server_name, uint32 switch_value,
                        WKS_INFO_100 *wks100);
 
+/*The following definitions come from  rpcclient/cmd_lsarpc.c  */
+
+void cmd_lsa_query_info(struct client_info *info);
+void cmd_lsa_lookup_sids(struct client_info *info);
+
+/*The following definitions come from  rpcclient/cmd_netlogon.c  */
+
+void cmd_netlogon_login_test(struct client_info *info);
+
+/*The following definitions come from  rpcclient/cmd_reg.c  */
+
+void cmd_reg_enum(struct client_info *info);
+void cmd_reg_query_key(struct client_info *info);
+void cmd_reg_create_val(struct client_info *info);
+void cmd_reg_delete_val(struct client_info *info);
+void cmd_reg_delete_key(struct client_info *info);
+void cmd_reg_create_key(struct client_info *info);
+void cmd_reg_test_key_sec(struct client_info *info);
+void cmd_reg_get_key_sec(struct client_info *info);
+
+/*The following definitions come from  rpcclient/cmd_samr.c  */
+
+void cmd_sam_ntchange_pwd(struct client_info *info);
+void cmd_sam_test(struct client_info *info);
+void cmd_sam_enum_users(struct client_info *info);
+void cmd_sam_query_user(struct client_info *info);
+void cmd_sam_query_groups(struct client_info *info);
+void cmd_sam_enum_aliases(struct client_info *info);
+
+/*The following definitions come from  rpcclient/cmd_srvsvc.c  */
+
+void cmd_srv_query_info(struct client_info *info);
+void cmd_srv_enum_conn(struct client_info *info);
+void cmd_srv_enum_shares(struct client_info *info);
+void cmd_srv_enum_sess(struct client_info *info);
+void cmd_srv_enum_files(struct client_info *info);
+
+/*The following definitions come from  rpcclient/cmd_wkssvc.c  */
+
+void cmd_wks_query_info(struct client_info *info);
+
+/*The following definitions come from  rpcclient/display.c  */
+
+char *get_file_mode_str(uint32 share_mode);
+char *get_file_oplock_str(uint32 op_type);
+char *get_share_type_str(uint32 type);
+char *get_server_type_str(uint32 type);
+void display_srv_info_101(FILE *out_hnd, enum action_type action,
+               SRV_INFO_101 *sv101);
+void display_srv_info_102(FILE *out_hnd, enum action_type action,SRV_INFO_102 *sv102);
+void display_srv_info_ctr(FILE *out_hnd, enum action_type action,SRV_INFO_CTR *ctr);
+void display_conn_info_0(FILE *out_hnd, enum action_type action,
+               CONN_INFO_0 *info0);
+void display_conn_info_1(FILE *out_hnd, enum action_type action,
+               CONN_INFO_1 *info1, CONN_INFO_1_STR *str1);
+void display_srv_conn_info_0_ctr(FILE *out_hnd, enum action_type action,
+                               SRV_CONN_INFO_0 *ctr);
+void display_srv_conn_info_1_ctr(FILE *out_hnd, enum action_type action,
+                               SRV_CONN_INFO_1 *ctr);
+void display_srv_conn_info_ctr(FILE *out_hnd, enum action_type action,
+                               SRV_CONN_INFO_CTR *ctr);
+void display_share_info_1(FILE *out_hnd, enum action_type action,
+                         SRV_SHARE_INFO_1 *info1);
+void display_share_info_2(FILE *out_hnd, enum action_type action,
+                         SRV_SHARE_INFO_2 *info2);
+void display_srv_share_info_ctr(FILE *out_hnd, enum action_type action,
+                               SRV_SHARE_INFO_CTR *ctr);
+void display_file_info_3(FILE *out_hnd, enum action_type action,
+               FILE_INFO_3 *info3, FILE_INFO_3_STR *str3);
+void display_srv_file_info_3_ctr(FILE *out_hnd, enum action_type action,
+                               SRV_FILE_INFO_3 *ctr);
+void display_srv_file_info_ctr(FILE *out_hnd, enum action_type action,
+                               SRV_FILE_INFO_CTR *ctr);
+void display_server(FILE *out_hnd, enum action_type action,
+                               char *sname, uint32 type, char *comment);
+void display_share(FILE *out_hnd, enum action_type action,
+                               char *sname, uint32 type, char *comment);
+void display_share2(FILE *out_hnd, enum action_type action,
+                               char *sname, uint32 type, char *comment,
+                               uint32 perms, uint32 max_uses, uint32 num_uses,
+                               char *path, char *passwd);
+void display_name(FILE *out_hnd, enum action_type action,
+                               char *sname);
+void display_group_rid_info(FILE *out_hnd, enum action_type action,
+                               uint32 num_gids, DOM_GID *gid);
+void display_alias_name_info(FILE *out_hnd, enum action_type action,
+                               uint32 num_aliases, fstring *alias_name, uint32 *num_als_usrs);
+void display_sam_user_info_21(FILE *out_hnd, enum action_type action, SAM_USER_INFO_21 *usr);
+char *get_sec_mask_str(uint32 type);
+void display_sec_access(FILE *out_hnd, enum action_type action, SEC_ACCESS *info);
+void display_sec_ace(FILE *out_hnd, enum action_type action, SEC_ACE *ace);
+void display_sec_acl(FILE *out_hnd, enum action_type action, SEC_ACL *sec_acl);
+void display_sec_desc(FILE *out_hnd, enum action_type action, SEC_DESC *sec);
+char *get_reg_val_type_str(uint32 type);
+void display_reg_value_info(FILE *out_hnd, enum action_type action,
+                               char *val_name, uint32 val_type, BUFFER2 *value);
+void display_reg_key_info(FILE *out_hnd, enum action_type action,
+                               char *key_name, time_t key_mod_time);
+
+/*The following definitions come from  rpcclient/rpcclient.c  */
+
+void rpcclient_init(void);
+
 /*The following definitions come from  rpc_parse/parse_creds.c  */
 
 BOOL make_creds_unix(CREDS_UNIX *r_u, const char* user_name,
@@ -2390,7 +2493,7 @@ BOOL spoolss_io_q_enumforms(char *desc, SPOOL_Q_ENUMFORMS *q_u, prs_struct *ps,
 BOOL new_spoolss_io_r_enumforms(char *desc, SPOOL_R_ENUMFORMS *r_u, prs_struct *ps, int depth);
 BOOL new_spoolss_io_r_enumports(char *desc, SPOOL_R_ENUMPORTS *r_u, prs_struct *ps, int depth);
 BOOL spoolss_io_q_enumports(char *desc, SPOOL_Q_ENUMPORTS *q_u, prs_struct *ps, int depth);
-BOOL spool_io_printer_info_level_2(char *desc, SPOOL_PRINTER_INFO_LEVEL_2 **q_u, prs_struct *ps, int depth);
+BOOL spool_io_printer_info_level_2(char *desc, SPOOL_PRINTER_INFO_LEVEL_2 *il, prs_struct *ps, int depth);
 BOOL spool_io_printer_info_level(char *desc, SPOOL_PRINTER_INFO_LEVEL *il, prs_struct *ps, int depth);
 BOOL spoolss_io_q_addprinterex(char *desc, SPOOL_Q_ADDPRINTEREX *q_u, prs_struct *ps, int depth);
 BOOL spoolss_io_r_addprinterex(char *desc, SPOOL_R_ADDPRINTEREX *r_u, prs_struct *ps, int depth);
@@ -2625,13 +2728,10 @@ uint32 _spoolss_writeprinter( const POLICY_HND *handle,
                                uint32 buffer_size,
                                const uint8 *buffer,
                                uint32 *buffer_written);
-uint32 _spoolss_setprinter( const POLICY_HND *handle,
-                               uint32 level,
-                               const SPOOL_PRINTER_INFO_LEVEL *info,
-                               const DEVICEMODE *devmode,
-                               uint32 sec_buf_size,
-                               const char *sec_buf,
-                               uint32 command);
+uint32 _spoolss_setprinter(const POLICY_HND *handle, uint32 level,
+                          const SPOOL_PRINTER_INFO_LEVEL *info,
+                          const DEVMODE_CTR devmode_ctr,
+                          uint32 command);
 uint32 _spoolss_fcpn(const POLICY_HND *handle);
 uint32 _spoolss_addjob(const POLICY_HND *handle, uint32 level,
                        NEW_BUFFER *buffer, uint32 offered);
@@ -2664,15 +2764,11 @@ uint32 _spoolss_addprinterdriver( const UNISTR2 *server_name,
 uint32 _spoolss_getprinterdriverdirectory(UNISTR2 *name, UNISTR2 *uni_environment, uint32 level,
                                        NEW_BUFFER *buffer, uint32 offered, 
                                        uint32 *needed);
-uint32 _spoolss_enumprinterdata(const POLICY_HND *handle, 
-                               uint32 idx,
-                               uint32 *valuesize,
-                               UNISTR *uni_value,
-                               uint32 *realvaluesize,
-                               uint32 *type,
-                               uint32 *datasize,
-                               uint8  **data,
-                               uint32 *realdatasize);
+uint32 _spoolss_enumprinterdata(const POLICY_HND *handle, uint32 index,
+                               uint32 in_value_len, uint32 in_data_len,
+                               uint32 *out_max_value_len, uint16 **out_value, uint32 *out_value_len,
+                               uint32 *out_type,
+                               uint32 *out_max_data_len, uint8  **out_data, uint32 *out_data_len);
 uint32 _spoolss_setprinterdata( const POLICY_HND *handle,
                                const UNISTR2 *value,
                                uint32 type,
@@ -2716,109 +2812,6 @@ uint32 lookup_user_rid(char *user_name, uint32 *rid);
 
 BOOL api_wkssvc_rpc(pipes_struct *p, prs_struct *data);
 
-/*The following definitions come from  rpcclient/cmd_lsarpc.c  */
-
-void cmd_lsa_query_info(struct client_info *info);
-void cmd_lsa_lookup_sids(struct client_info *info);
-
-/*The following definitions come from  rpcclient/cmd_netlogon.c  */
-
-void cmd_netlogon_login_test(struct client_info *info);
-
-/*The following definitions come from  rpcclient/cmd_reg.c  */
-
-void cmd_reg_enum(struct client_info *info);
-void cmd_reg_query_key(struct client_info *info);
-void cmd_reg_create_val(struct client_info *info);
-void cmd_reg_delete_val(struct client_info *info);
-void cmd_reg_delete_key(struct client_info *info);
-void cmd_reg_create_key(struct client_info *info);
-void cmd_reg_test_key_sec(struct client_info *info);
-void cmd_reg_get_key_sec(struct client_info *info);
-
-/*The following definitions come from  rpcclient/cmd_samr.c  */
-
-void cmd_sam_ntchange_pwd(struct client_info *info);
-void cmd_sam_test(struct client_info *info);
-void cmd_sam_enum_users(struct client_info *info);
-void cmd_sam_query_user(struct client_info *info);
-void cmd_sam_query_groups(struct client_info *info);
-void cmd_sam_enum_aliases(struct client_info *info);
-
-/*The following definitions come from  rpcclient/cmd_srvsvc.c  */
-
-void cmd_srv_query_info(struct client_info *info);
-void cmd_srv_enum_conn(struct client_info *info);
-void cmd_srv_enum_shares(struct client_info *info);
-void cmd_srv_enum_sess(struct client_info *info);
-void cmd_srv_enum_files(struct client_info *info);
-
-/*The following definitions come from  rpcclient/cmd_wkssvc.c  */
-
-void cmd_wks_query_info(struct client_info *info);
-
-/*The following definitions come from  rpcclient/display.c  */
-
-char *get_file_mode_str(uint32 share_mode);
-char *get_file_oplock_str(uint32 op_type);
-char *get_share_type_str(uint32 type);
-char *get_server_type_str(uint32 type);
-void display_srv_info_101(FILE *out_hnd, enum action_type action,
-               SRV_INFO_101 *sv101);
-void display_srv_info_102(FILE *out_hnd, enum action_type action,SRV_INFO_102 *sv102);
-void display_srv_info_ctr(FILE *out_hnd, enum action_type action,SRV_INFO_CTR *ctr);
-void display_conn_info_0(FILE *out_hnd, enum action_type action,
-               CONN_INFO_0 *info0);
-void display_conn_info_1(FILE *out_hnd, enum action_type action,
-               CONN_INFO_1 *info1, CONN_INFO_1_STR *str1);
-void display_srv_conn_info_0_ctr(FILE *out_hnd, enum action_type action,
-                               SRV_CONN_INFO_0 *ctr);
-void display_srv_conn_info_1_ctr(FILE *out_hnd, enum action_type action,
-                               SRV_CONN_INFO_1 *ctr);
-void display_srv_conn_info_ctr(FILE *out_hnd, enum action_type action,
-                               SRV_CONN_INFO_CTR *ctr);
-void display_share_info_1(FILE *out_hnd, enum action_type action,
-                         SRV_SHARE_INFO_1 *info1);
-void display_share_info_2(FILE *out_hnd, enum action_type action,
-                         SRV_SHARE_INFO_2 *info2);
-void display_srv_share_info_ctr(FILE *out_hnd, enum action_type action,
-                               SRV_SHARE_INFO_CTR *ctr);
-void display_file_info_3(FILE *out_hnd, enum action_type action,
-               FILE_INFO_3 *info3, FILE_INFO_3_STR *str3);
-void display_srv_file_info_3_ctr(FILE *out_hnd, enum action_type action,
-                               SRV_FILE_INFO_3 *ctr);
-void display_srv_file_info_ctr(FILE *out_hnd, enum action_type action,
-                               SRV_FILE_INFO_CTR *ctr);
-void display_server(FILE *out_hnd, enum action_type action,
-                               char *sname, uint32 type, char *comment);
-void display_share(FILE *out_hnd, enum action_type action,
-                               char *sname, uint32 type, char *comment);
-void display_share2(FILE *out_hnd, enum action_type action,
-                               char *sname, uint32 type, char *comment,
-                               uint32 perms, uint32 max_uses, uint32 num_uses,
-                               char *path, char *passwd);
-void display_name(FILE *out_hnd, enum action_type action,
-                               char *sname);
-void display_group_rid_info(FILE *out_hnd, enum action_type action,
-                               uint32 num_gids, DOM_GID *gid);
-void display_alias_name_info(FILE *out_hnd, enum action_type action,
-                               uint32 num_aliases, fstring *alias_name, uint32 *num_als_usrs);
-void display_sam_user_info_21(FILE *out_hnd, enum action_type action, SAM_USER_INFO_21 *usr);
-char *get_sec_mask_str(uint32 type);
-void display_sec_access(FILE *out_hnd, enum action_type action, SEC_ACCESS *info);
-void display_sec_ace(FILE *out_hnd, enum action_type action, SEC_ACE *ace);
-void display_sec_acl(FILE *out_hnd, enum action_type action, SEC_ACL *sec_acl);
-void display_sec_desc(FILE *out_hnd, enum action_type action, SEC_DESC *sec);
-char *get_reg_val_type_str(uint32 type);
-void display_reg_value_info(FILE *out_hnd, enum action_type action,
-                               char *val_name, uint32 val_type, BUFFER2 *value);
-void display_reg_key_info(FILE *out_hnd, enum action_type action,
-                               char *key_name, time_t key_mod_time);
-
-/*The following definitions come from  rpcclient/rpcclient.c  */
-
-void rpcclient_init(void);
-
 /*The following definitions come from  smbd/blocking.c  */
 
 BOOL push_blocking_lock_request( char *inbuf, int length, int lock_timeout, int lock_num);
@@ -3204,6 +3197,19 @@ BOOL unbecome_authenticated_pipe_user(pipes_struct *p);
 void become_root(BOOL save_dir) ;
 void unbecome_root(BOOL restore_dir);
 
+/*The following definitions come from  smbd/vfs.c  */
+
+int vfs_init_default(connection_struct *conn);
+BOOL vfs_init_custom(connection_struct *conn);
+BOOL vfs_directory_exist(connection_struct *conn, char *dname,
+                         SMB_STRUCT_STAT *st);
+BOOL vfs_file_exist(connection_struct *conn,char *fname,SMB_STRUCT_STAT *sbuf);
+ssize_t vfs_write_data(files_struct *fsp,char *buffer,size_t N);
+SMB_OFF_T vfs_transfer_file(int in_fd, files_struct *in_fsp, 
+                           int out_fd, files_struct *out_fsp,
+                           SMB_OFF_T n, char *header, int headlen, int align);
+char *vfs_readdirname(connection_struct *conn, void *p);
+
 /*The following definitions come from  smbd/vfs-wrap.c  */
 
 int vfswrap_dummy_connect(struct vfs_connection_struct *conn, char *service,
@@ -3231,19 +3237,6 @@ int vfswrap_unlink(char *path);
 int vfswrap_chmod(char *path, mode_t mode);
 int vfswrap_utime(char *path, struct utimbuf *times);
 
-/*The following definitions come from  smbd/vfs.c  */
-
-int vfs_init_default(connection_struct *conn);
-BOOL vfs_init_custom(connection_struct *conn);
-BOOL vfs_directory_exist(connection_struct *conn, char *dname,
-                         SMB_STRUCT_STAT *st);
-BOOL vfs_file_exist(connection_struct *conn,char *fname,SMB_STRUCT_STAT *sbuf);
-ssize_t vfs_write_data(files_struct *fsp,char *buffer,size_t N);
-SMB_OFF_T vfs_transfer_file(int in_fd, files_struct *in_fsp, 
-                           int out_fd, files_struct *out_fsp,
-                           SMB_OFF_T n, char *header, int headlen, int align);
-char *vfs_readdirname(connection_struct *conn, void *p);
-
 /*The following definitions come from  smbwrapper/realcalls.c  */
 
 int real_utime(const char *name, struct utimbuf *buf);
index 19b6c2845ced6dc6ffb2dd86fcb5f3a3e87295f3..84812ed98a62724a40eaa64d29d89e18f921cea4 100755 (executable)
@@ -1138,6 +1138,8 @@ typedef struct spool_printer_info_level_2
 
 typedef struct spool_printer_info_level
 {
+       uint32 level;
+       uint32 info_ptr;
        SPOOL_PRINTER_INFO_LEVEL_2 *info_2;
 } SPOOL_PRINTER_INFO_LEVEL;
 
@@ -1200,8 +1202,7 @@ typedef struct spool_q_setprinter
        POLICY_HND handle;
        uint32 level;
        SPOOL_PRINTER_INFO_LEVEL info;
-
-       DEVICEMODE *devmode;
+       DEVMODE_CTR devmode_ctr;
 
        /* lkclXXXX jean-francois, see SEC_DESC_BUF code */
        struct
@@ -1391,7 +1392,7 @@ typedef struct spool_q_enumprinterdata
 typedef struct spool_r_enumprinterdata
 {
        uint32 valuesize;
-       UNISTR value;
+       uint16 *value;
        uint32 realvaluesize;
        uint32 type;
        uint32 datasize;
index 0979799a98bf10a7e833c25407facbb752c83e64..d1f7ed61a405761d832d32cf98a988c022e0b4c2 100644 (file)
@@ -575,6 +575,9 @@ static BOOL spoolss_io_devmode_cont(char *desc, DEVMODE_CTR *dm_c, prs_struct *p
        prs_debug(ps, depth, desc, "spoolss_io_devmode_cont");
        depth++;
 
+       if(!prs_align(ps))
+               return False;
+       
        if (!prs_uint32("size", ps, depth, &dm_c->size))
                return False;
 
@@ -1164,8 +1167,9 @@ static uint32 size_of_systemtime(SYSTEMTIME *systime)
  * write a UNICODE string.
  * used by all the RPC structs passing a buffer
  ********************************************************************/
-static BOOL spoolss_smb_io_unistr(char *desc,  UNISTR *uni, prs_struct *ps, int depth)
+static BOOL spoolss_smb_io_unistr(char *desc, UNISTR *uni, prs_struct *ps, int depth)
 {
+       uint16 zero=0;
        if (uni == NULL)
                return False;
 
@@ -1173,6 +1177,8 @@ static BOOL spoolss_smb_io_unistr(char *desc,  UNISTR *uni, prs_struct *ps, int
        depth++;
        if (!prs_unistr("unistr", ps, depth, uni))
                return False;
+       if (!prs_uint16("null", ps, depth, &zero))
+               return False;
 }
 
 
@@ -2645,24 +2651,25 @@ BOOL spoolss_io_q_setprinter(char *desc, SPOOL_Q_SETPRINTER *q_u, prs_struct *ps
        prs_debug(ps, depth, desc, "spoolss_io_q_setprinter");
        depth++;
 
-       prs_align(ps);
-
-       smb_io_prt_hnd("printer handle", &(q_u->handle),ps,depth);
-       prs_uint32("level", ps, depth, &(q_u->level));
-
-       /* again a designed mess */
-       /* sometimes I'm wondering how all of this work ! */
+       if(!prs_align(ps))
+               return False;
 
-       /* To be correct it need to be split in 3 functions */
+       if(!smb_io_prt_hnd("printer handle", &q_u->handle ,ps, depth))
+               return False;
+       if(!prs_uint32("level", ps, depth, &q_u->level))
+               return False;
 
-       spool_io_printer_info_level("", &(q_u->info), ps, depth);
+       if(!spool_io_printer_info_level("", &q_u->info, ps, depth))
+               return False;
 
-       spoolss_io_devmode(desc, ps, depth, q_u->devmode);
+       if (!spoolss_io_devmode_cont(desc, &q_u->devmode_ctr, ps, depth))
+               return False;
        
-       prs_uint32("security.size_of_buffer", ps, depth, &(q_u->security.size_of_buffer));
-       prs_uint32("security.data",           ps, depth, &(q_u->security.data));
+       prs_uint32("security.size_of_buffer", ps, depth, &q_u->security.size_of_buffer);
+       prs_uint32("security.data", ps, depth, &q_u->security.data);
        
-       prs_uint32("command", ps, depth, &(q_u->command));
+       if(!prs_uint32("command", ps, depth, &q_u->command))
+               return False;
 
        return True;
 }
@@ -3074,77 +3081,83 @@ BOOL spoolss_io_q_enumports(char *desc, SPOOL_Q_ENUMPORTS *q_u, prs_struct *ps,
        return True;
 }
 
-
 /*******************************************************************
+ Parse a SPOOL_PRINTER_INFO_LEVEL_2 structure.
 ********************************************************************/  
-BOOL spool_io_printer_info_level_2(char *desc, SPOOL_PRINTER_INFO_LEVEL_2 **q_u, prs_struct *ps, int depth)
+BOOL spool_io_printer_info_level_2(char *desc, SPOOL_PRINTER_INFO_LEVEL_2 *il, prs_struct *ps, int depth)
 {      
-       SPOOL_PRINTER_INFO_LEVEL_2 *il;
-       
-       prs_debug(ps, depth, desc, "");
+       prs_debug(ps, depth, desc, "spool_io_printer_info_level_2");
        depth++;
-
-       /* reading */
-       if (ps->io)
-       {
-               il=(SPOOL_PRINTER_INFO_LEVEL_2 *)malloc(sizeof(SPOOL_PRINTER_INFO_LEVEL_2));
-               ZERO_STRUCTP(il);
-               *q_u=il;
-               DEBUG(7,("lecture: memoire ok\n"));
-       }
-       else
-       {
-               il=*q_u;
-       }
                
-       prs_align(ps);  
-
-       prs_uint32("servername_ptr",     ps, depth, &(il->servername_ptr));
-       prs_uint32("printername_ptr",    ps, depth, &(il->printername_ptr));
-       prs_uint32("sharename_ptr",      ps, depth, &(il->sharename_ptr));
-       prs_uint32("portname_ptr",       ps, depth, &(il->portname_ptr));
-       prs_uint32("drivername_ptr",     ps, depth, &(il->drivername_ptr));
-       prs_uint32("comment_ptr",        ps, depth, &(il->comment_ptr));
-       prs_uint32("location_ptr",       ps, depth, &(il->location_ptr));
-       prs_uint32("devmode_ptr",        ps, depth, &(il->devmode_ptr));
-       prs_uint32("sepfile_ptr",        ps, depth, &(il->sepfile_ptr));
-       prs_uint32("printprocessor_ptr", ps, depth, &(il->printprocessor_ptr));
-       prs_uint32("datatype_ptr",       ps, depth, &(il->datatype_ptr));
-       prs_uint32("parameters_ptr",     ps, depth, &(il->parameters_ptr));
-       prs_uint32("secdesc_ptr",        ps, depth, &(il->secdesc_ptr));
-
-       prs_uint32("attributes",         ps, depth, &(il->attributes));
-       prs_uint32("priority",           ps, depth, &(il->priority));
-       prs_uint32("default_priority",   ps, depth, &(il->default_priority));
-       prs_uint32("starttime",          ps, depth, &(il->starttime));
-       prs_uint32("untiltime",          ps, depth, &(il->untiltime));
-       prs_uint32("status",             ps, depth, &(il->status));
-       prs_uint32("cjobs",              ps, depth, &(il->cjobs));
-       prs_uint32("averageppm",         ps, depth, &(il->averageppm));
-
-       smb_io_unistr2("", &(il->servername),     il->servername_ptr,     ps, depth);   
-       smb_io_unistr2("", &(il->printername),    il->printername_ptr,    ps, depth);   
-       smb_io_unistr2("", &(il->sharename),      il->sharename_ptr,      ps, depth);   
-       smb_io_unistr2("", &(il->portname),       il->portname_ptr,       ps, depth);   
-       smb_io_unistr2("", &(il->drivername),     il->drivername_ptr,     ps, depth);   
-       smb_io_unistr2("", &(il->comment),        il->comment_ptr,        ps, depth);   
-       smb_io_unistr2("", &(il->location),       il->location_ptr,       ps, depth);   
-       smb_io_unistr2("", &(il->sepfile),        il->sepfile_ptr,        ps, depth);   
-       smb_io_unistr2("", &(il->printprocessor), il->printprocessor_ptr, ps, depth);   
-       smb_io_unistr2("", &(il->datatype),       il->datatype_ptr,       ps, depth);   
-       smb_io_unistr2("", &(il->parameters),     il->parameters_ptr,     ps, depth);   
+       if(!prs_align(ps))
+               return False;
 
-       prs_align(ps);
+       if(!prs_uint32("servername_ptr", ps, depth, &il->servername_ptr))
+               return False;
+       if(!prs_uint32("printername_ptr", ps, depth, &il->printername_ptr))
+               return False;
+       if(!prs_uint32("sharename_ptr", ps, depth, &il->sharename_ptr))
+               return False;
+       if(!prs_uint32("portname_ptr", ps, depth, &il->portname_ptr))
+               return False;
+       if(!prs_uint32("drivername_ptr", ps, depth, &il->drivername_ptr))
+               return False;
+       if(!prs_uint32("comment_ptr", ps, depth, &il->comment_ptr))
+               return False;
+       if(!prs_uint32("location_ptr", ps, depth, &il->location_ptr))
+               return False;
+       if(!prs_uint32("devmode_ptr", ps, depth, &il->devmode_ptr))
+               return False;
+       if(!prs_uint32("sepfile_ptr", ps, depth, &il->sepfile_ptr))
+               return False;
+       if(!prs_uint32("printprocessor_ptr", ps, depth, &il->printprocessor_ptr))
+               return False;
+       if(!prs_uint32("datatype_ptr", ps, depth, &il->datatype_ptr))
+               return False;
+       if(!prs_uint32("parameters_ptr", ps, depth, &il->parameters_ptr))
+               return False;
+       if(!prs_uint32("secdesc_ptr", ps, depth, &il->secdesc_ptr))
+               return False;
 
-       /* this code as nothing to do here !!!
-       
-       if (il->secdesc_ptr)
-       {
-               il->secdesc=NULL;
-               sec_io_desc_buf("", &(il->secdesc), ps, depth);
-       }
-       
-       */
+       if(!prs_uint32("attributes", ps, depth, &il->attributes))
+               return False;
+       if(!prs_uint32("priority", ps, depth, &il->priority))
+               return False;
+       if(!prs_uint32("default_priority", ps, depth, &il->default_priority))
+               return False;
+       if(!prs_uint32("starttime", ps, depth, &il->starttime))
+               return False;
+       if(!prs_uint32("untiltime", ps, depth, &il->untiltime))
+               return False;
+       if(!prs_uint32("status", ps, depth, &il->status))
+               return False;
+       if(!prs_uint32("cjobs", ps, depth, &il->cjobs))
+               return False;
+       if(!prs_uint32("averageppm", ps, depth, &il->averageppm))
+               return False;
+
+       if(!smb_io_unistr2("servername", &il->servername, il->servername_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("printername", &il->printername, il->printername_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("sharename", &il->sharename, il->sharename_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("portname", &il->portname, il->portname_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("drivername", &il->drivername, il->drivername_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("comment", &il->comment, il->comment_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("location", &il->location, il->location_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("sepfile", &il->sepfile, il->sepfile_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("printprocessor", &il->printprocessor, il->printprocessor_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("datatype", &il->datatype, il->datatype_ptr, ps, depth))
+               return False;
+       if(!smb_io_unistr2("parameters", &il->parameters, il->parameters_ptr, ps, depth))
+               return False;
 
        return True;
 }
@@ -3153,16 +3166,24 @@ BOOL spool_io_printer_info_level_2(char *desc, SPOOL_PRINTER_INFO_LEVEL_2 **q_u,
 ********************************************************************/  
 BOOL spool_io_printer_info_level(char *desc, SPOOL_PRINTER_INFO_LEVEL *il, prs_struct *ps, int depth)
 {
-       uint32 useless;
-       uint32 level;
-       prs_debug(ps, depth, desc, "");
+       prs_debug(ps, depth, desc, "spool_io_printer_info_level");
        depth++;
 
-       prs_align(ps);  
-       prs_uint32("info level", ps, depth, &level);
-       prs_uint32("useless", ps, depth, &useless);
-               
-       switch (level) {
+       if(!prs_align(ps))
+               return False;
+       if(!prs_uint32("level", ps, depth, &il->level))
+               return False;
+       if(!prs_uint32("info_ptr", ps, depth, &il->info_ptr))
+               return False;
+       
+       /* if no struct inside just return */
+       if (il->info_ptr==0) {
+               if (UNMARSHALLING(ps))
+                       il->info_2=NULL;
+               return True;
+       }
+                       
+       switch (il->level) {
                /*
                 * level 0 is used by setprinter when managing the queue
                 * (hold, stop, start a queue)
@@ -3174,11 +3195,13 @@ BOOL spool_io_printer_info_level(char *desc, SPOOL_PRINTER_INFO_LEVEL *il, prs_s
                 * and by setprinter when updating printer's info
                 */     
                case 2:
-                       spool_io_printer_info_level_2("", &(il->info_2), ps, depth);
+                       if (UNMARSHALLING(ps))
+                               il->info_2=(SPOOL_PRINTER_INFO_LEVEL_2 *)malloc(sizeof(SPOOL_PRINTER_INFO_LEVEL_2));
+                       if (!spool_io_printer_info_level_2("", il->info_2, ps, depth))
+                               return False;
                        break;          
        }
 
-
        return True;
 }
 
@@ -3761,17 +3784,26 @@ BOOL spoolss_io_r_enumprinterdata(char *desc, SPOOL_R_ENUMPRINTERDATA *r_u, prs_
        prs_debug(ps, depth, desc, "spoolss_io_r_enumprinterdata");
        depth++;
 
-       prs_align(ps);  
-       prs_uint32("valuesize",     ps, depth, &(r_u->valuesize));
-       prs_unistr("value",         ps, depth, &(r_u->value));
-       prs_uint32("realvaluesize", ps, depth, &(r_u->realvaluesize));
+       if(!prs_align(ps))
+               return False;
+       if(!prs_uint32("valuesize", ps, depth, &r_u->valuesize))
+               return False;
+       if(!prs_uint16s(False, "value", ps, depth, r_u->value, r_u->valuesize))
+               return False;
+       if(!prs_uint32("realvaluesize", ps, depth, &r_u->realvaluesize))
+               return False;
 
-       prs_uint32("type",          ps, depth, &(r_u->type));
+       if(!prs_uint32("type", ps, depth, &r_u->type))
+               return False;
 
-       prs_uint32("datasize",      ps, depth, &(r_u->datasize));
-       prs_uint8s(False, "data",   ps, depth, r_u->data, r_u->datasize);
-       prs_uint32("realdatasize",  ps, depth, &(r_u->realdatasize));
-       prs_uint32("status",        ps, depth, &(r_u->status));
+       if(!prs_uint32("datasize", ps, depth, &r_u->datasize))
+               return False;
+       if(!prs_uint8s(False, "data", ps, depth, r_u->data, r_u->datasize))
+               return False;
+       if(!prs_uint32("realdatasize", ps, depth, &r_u->realdatasize))
+               return False;
+       if(!prs_uint32("status", ps, depth, &r_u->status))
+               return False;
 
        return True;
 }
@@ -3783,11 +3815,16 @@ BOOL spoolss_io_q_enumprinterdata(char *desc, SPOOL_Q_ENUMPRINTERDATA *q_u, prs_
        prs_debug(ps, depth, desc, "spoolss_io_q_enumprinterdata");
        depth++;
 
-       prs_align(ps);
-       smb_io_prt_hnd("printer handle",&(q_u->handle),ps,depth);
-       prs_uint32("index",     ps, depth, &(q_u->index));
-       prs_uint32("valuesize", ps, depth, &(q_u->valuesize));
-       prs_uint32("datasize",  ps, depth, &(q_u->datasize));
+       if(!prs_align(ps))
+               return False;
+       if(!smb_io_prt_hnd("printer handle",&q_u->handle,ps,depth))
+               return False;
+       if(!prs_uint32("index", ps, depth, &q_u->index))
+               return False;
+       if(!prs_uint32("valuesize", ps, depth, &q_u->valuesize))
+               return False;
+       if(!prs_uint32("datasize", ps, depth, &q_u->datasize))
+               return False;
 
        return True;
 }
index be6775f35d2e89b2f61c2075d724773f364ac118..d20f6c1616f0542a36240da5d18ea482b01614f8 100755 (executable)
@@ -444,8 +444,6 @@ static BOOL api_spoolss_writeprinter(uint16 vuid, prs_struct *data, prs_struct *
 
 /****************************************************************************
 
-FIX ME: JFM: freeing memory ????
-
 ****************************************************************************/
 static BOOL api_spoolss_setprinter(uint16 vuid, prs_struct *data, prs_struct *rdata)
 {
@@ -460,13 +458,15 @@ static BOOL api_spoolss_setprinter(uint16 vuid, prs_struct *data, prs_struct *rd
                return False;
        }
        
-       DEBUG(0,("api_spoolss_setprinter: typecast sec_des to uint8*!\n"));
-       r_u.status = _spoolss_setprinter(&q_u.handle,
-                                        q_u.level, &q_u.info,
-                                        q_u.devmode,
-                                        q_u.security.size_of_buffer,
-                                        (const uint8*)q_u.security.data,
-                                        q_u.command);
+       r_u.status = _spoolss_setprinter(&q_u.handle, q_u.level, &q_u.info,
+                                        q_u.devmode_ctr, q_u.command);
+       
+       /* now, we can free the memory */
+       if (q_u.info.level==2 && q_u.info.info_ptr!=0)
+               safe_free(q_u.info.info_2);
+               
+       if (q_u.devmode_ctr.devmode_ptr!=0)
+               safe_free(q_u.devmode_ctr.devmode);
        
        if(!spoolss_io_r_setprinter("",&r_u,rdata,0)) {
                DEBUG(0,("spoolss_io_r_setprinter: unable to marshall SPOOL_R_SETPRINTER.\n"));
@@ -836,24 +836,19 @@ static BOOL api_spoolss_enumprinterdata(uint16 vuid, prs_struct *data, prs_struc
                return False;
        }
        
-       r_u.valuesize = q_u.valuesize;
-       r_u.datasize = q_u.datasize;
-
-       r_u.status = _spoolss_enumprinterdata(&q_u.handle,
-                               q_u.index,/* in */
-                               &r_u.valuesize,/* in out */
-                               &r_u.value,/* out */
-                               &r_u.realvaluesize,/* out */
-                               &r_u.type,/* out */
-                               &r_u.datasize,/* in out */
-                               &r_u.data,/* out */
-                               &r_u.realdatasize);/* out */
+       r_u.status = _spoolss_enumprinterdata(&q_u.handle, q_u.index, q_u.valuesize, q_u.datasize,
+                                               &r_u.valuesize, &r_u.value, &r_u.realvaluesize,
+                                               &r_u.type,
+                                               &r_u.datasize, &r_u.data, &r_u.realdatasize);
                                
        if(!spoolss_io_r_enumprinterdata("", &r_u, rdata, 0)) {
                DEBUG(0,("spoolss_io_r_enumprinterdata: unable to marshall SPOOL_R_ENUMPRINTERDATA.\n"));
+               safe_free(r_u.value);
+               safe_free(r_u.data);
                return False;
        }
 
+       safe_free(r_u.value);
        safe_free(r_u.data);
 
        return True;
index da7eda57958d42912ea5d06cc637e2ae8d8cbb83..3ab426e9c2cd0e68fbc08e39ca3262a2091ee497 100644 (file)
@@ -66,6 +66,10 @@ typedef struct _Printer{
                uint32 printerlocal;
                SPOOL_NOTIFY_OPTION *option;
        } notify;
+       struct {
+               fstring machine;
+               fstring user;
+       } client;
 } Printer_entry;
 
 static ubi_dlList Printer_list;
@@ -160,8 +164,6 @@ static BOOL close_printer_handle(POLICY_HND *hnd)
 
        safe_free(Printer);
 
-       DEBUG(0,("[%d] entrys still in list\n", ubi_dlCount(&Printer_list)));
-
        return True;
 }      
 
@@ -2632,25 +2634,30 @@ static uint32 control_printer(const POLICY_HND *handle, uint32 command)
        if (!OPEN_HANDLE(Printer))
                return NT_STATUS_INVALID_HANDLE;
 
-       if (!get_printer_snum(handle, &snum) )
-       {                
+       if (!get_printer_snum(handle, &snum) )   
                return NT_STATUS_INVALID_HANDLE;
-       }
 
-       switch (command)
-       {
+       switch (command) {
                case PRINTER_CONTROL_PAUSE:
                        /* pause the printer here */
                        status_printqueue(NULL, snum, LPSTAT_STOPPED);
                        return 0x0;
-
+                       break;
                case PRINTER_CONTROL_RESUME:
                case PRINTER_CONTROL_UNPAUSE:
                        /* UN-pause the printer here */
                        status_printqueue(NULL, snum, LPSTAT_OK);
                        return 0x0;
+                       break;
                case PRINTER_CONTROL_PURGE:
-                       /* Envoi des dragées FUCA dans l'imprimante */
+                       /*
+                        * It's not handled by samba
+                        * we need a smb.conf param to do
+                        * lprm -P%p - on BSD
+                        * lprm -P%p all on LPRNG
+                        * I don't know on SysV
+                        * we could do it by looping in the job's list...
+                        */
                        break;
        }
 
@@ -2675,9 +2682,8 @@ static uint32 update_printer(const POLICY_HND *handle, uint32 level,
        
        DEBUG(8,("update_printer\n"));
        
-       if (level!=2)
-       {
-               DEBUG(0,("Send a mail to samba-bugs@samba.org\n"));
+       if (level!=2) {
+               DEBUG(0,("Send a mail to jfm@samba.org\n"));
                DEBUGADD(0,("with the following message: update_printer: level!=2\n"));
                return NT_STATUS_INVALID_INFO_CLASS;
        }
@@ -2688,13 +2694,12 @@ static uint32 update_printer(const POLICY_HND *handle, uint32 level,
        if (!get_printer_snum(handle, &snum) )
                return NT_STATUS_INVALID_HANDLE;
        
-       get_a_printer(&printer, level, lp_servicename(snum));
+       get_a_printer(&printer, 2, lp_servicename(snum));
 
        DEBUGADD(8,("Converting info_2 struct\n"));
        convert_printer_info(info, &printer, level);
        
-       if ((info->info_2)->devmode_ptr != 0)
-       {
+       if ((info->info_2)->devmode_ptr != 0) {
                /* we have a valid devmode
                   convert it and link it*/
                
@@ -2710,36 +2715,30 @@ static uint32 update_printer(const POLICY_HND *handle, uint32 level,
                                
                convert_devicemode(*devmode, nt_devmode);
        }
-       else
-       {
+       else {
                if (printer.info_2->devmode != NULL)
-               {
                        free(printer.info_2->devmode);
-               }
                printer.info_2->devmode=NULL;
        }
                        
-       if (status == 0x0)
-       {
-               status = add_a_printer(printer, level);
-       }
-       if (status == 0x0)
-       {
-               status = free_a_printer(printer, level);
+       if (add_a_printer(printer, 2)!=0) {
+               free_a_printer(printer, 2);
+               
+               /* I don't really know what to return here !!! */
+               return NT_STATUS_INVALID_INFO_CLASS;
        }
 
-       return status;
+       free_a_printer(printer, 2);
+
+       return NT_STATUS_NO_PROBLEMO;
 }
 
 /****************************************************************************
 ****************************************************************************/
-uint32 _spoolss_setprinter( const POLICY_HND *handle,
-                               uint32 level,
-                               const SPOOL_PRINTER_INFO_LEVEL *info,
-                               const DEVICEMODE *devmode,
-                               uint32 sec_buf_size,
-                               const char *sec_buf,
-                               uint32 command)
+uint32 _spoolss_setprinter(const POLICY_HND *handle, uint32 level,
+                          const SPOOL_PRINTER_INFO_LEVEL *info,
+                          const DEVMODE_CTR devmode_ctr,
+                          uint32 command)
 {
        Printer_entry *Printer = find_printer_index_by_hnd(handle);
        
@@ -2747,13 +2746,12 @@ uint32 _spoolss_setprinter( const POLICY_HND *handle,
                return NT_STATUS_INVALID_HANDLE;
 
        /* check the level */   
-       switch (level)
-       {
+       switch (level) {
                case 0:
                        return control_printer(handle, command);
                        break;
                case 2:
-                       return update_printer(handle, level, info, devmode);
+                       return update_printer(handle, level, info, devmode_ctr.devmode);
                        break;
        }
 
@@ -3094,7 +3092,7 @@ static uint32 enumprinterdrivers_level1(fstring *list, fstring servername, fstri
        /* fill the buffer with the form structures */
        for (i=0; i<*returned; i++)
        {
-               DEBUGADD(6,("adding form [%d] to buffer\n",i));
+               DEBUGADD(6,("adding driver [%d] to buffer\n",i));
                new_smb_io_printer_driver_info_1("", buffer, &(driver_info_1[i]), 0);
        }
 
@@ -3135,7 +3133,7 @@ static uint32 enumprinterdrivers_level2(fstring *list, fstring servername, fstri
        /* fill the buffer with the form structures */
        for (i=0; i<*returned; i++)
        {
-               DEBUGADD(6,("adding form [%d] to buffer\n",i));
+               DEBUGADD(6,("adding driver [%d] to buffer\n",i));
                new_smb_io_printer_driver_info_2("", buffer, &(driver_info_2[i]), 0);
        }
 
@@ -3588,15 +3586,11 @@ uint32 _spoolss_getprinterdriverdirectory(UNISTR2 *name, UNISTR2 *uni_environmen
        
 /****************************************************************************
 ****************************************************************************/
-uint32 _spoolss_enumprinterdata(const POLICY_HND *handle, 
-                               uint32 idx,
-                               uint32 *valuesize,
-                               UNISTR *uni_value,
-                               uint32 *realvaluesize,
-                               uint32 *type,
-                               uint32 *datasize,
-                               uint8  **data,
-                               uint32 *realdatasize)
+uint32 _spoolss_enumprinterdata(const POLICY_HND *handle, uint32 index,
+                               uint32 in_value_len, uint32 in_data_len,
+                               uint32 *out_max_value_len, uint16 **out_value, uint32 *out_value_len,
+                               uint32 *out_type,
+                               uint32 *out_max_data_len, uint8  **out_data, uint32 *out_data_len)
 {
        NT_PRINTER_INFO_LEVEL printer;
        
@@ -3606,12 +3600,22 @@ uint32 _spoolss_enumprinterdata(const POLICY_HND *handle,
        uint32 biggest_valuesize;
        uint32 biggest_datasize;
        uint32 data_len;
-       uint32 status = 0x0;    
        Printer_entry *Printer = find_printer_index_by_hnd(handle);
        int snum;
+       uint8 *data=NULL;
+       uint32 type;
 
        ZERO_STRUCT(printer);
-       (*data)=NULL;
+       
+       *out_max_value_len=0;
+       *out_value=NULL;
+       *out_value_len=0;
+
+       *out_type=0;
+
+       *out_max_data_len=0;
+       *out_data=NULL;
+       *out_data_len=0;
 
        DEBUG(5,("spoolss_enumprinterdata\n"));
 
@@ -3621,74 +3625,77 @@ uint32 _spoolss_enumprinterdata(const POLICY_HND *handle,
        if (!get_printer_snum(handle, &snum))
                return NT_STATUS_INVALID_HANDLE;
        
-       status = get_a_printer(&printer, 2, lp_servicename(snum));
-
-       if (status != 0x0)
-               return status;
+       if (get_a_printer(&printer, 2, lp_servicename(snum)) != 0x0)
+               return NT_STATUS_INVALID_HANDLE;
 
-       /* The NT machine wants to know the biggest size of value and data */   
-       if ( ((*valuesize)==0) && ((*datasize)==0) )
-       {
+       /* 
+        * The NT machine wants to know the biggest size of value and data
+        *
+        * cf: MSDN EnumPrinterData remark section
+        */
+       if ( (in_value_len==0) && (in_data_len==0) ) {
                DEBUGADD(6,("Activating NT mega-hack to find sizes\n"));
                
-               (*valuesize)=0;
-               (*realvaluesize)=0;
-               (*type)=0;
-               (*datasize)=0;
-               (*realdatasize)=0;
-               status=0;
-               
                param_index=0;
                biggest_valuesize=0;
                biggest_datasize=0;
                
-               while (get_specific_param_by_index(printer, 2, param_index, value, data, type, &data_len))
-               {
+               while (get_specific_param_by_index(printer, 2, param_index, value, &data, &type, &data_len)) {
                        if (strlen(value) > biggest_valuesize) biggest_valuesize=strlen(value);
-                       if (data_len  > biggest_datasize)  biggest_datasize=data_len;
+                       if (data_len > biggest_datasize) biggest_datasize=data_len;
 
+                       DEBUG(6,("current values: [%d], [%d]\n", biggest_valuesize, biggest_datasize));
+
+                       safe_free(data);
                        param_index++;
                }
-               
-               /* I wrote it, I didn't designed the protocol */
-               if (biggest_valuesize!=0)
-               {
-                       SIVAL(&(value),0, 2*(biggest_valuesize+1) );
-               }
-               (*data)=(uint8 *)malloc(4*sizeof(uint8));
-               SIVAL((*data), 0, biggest_datasize );
+
+               /* the value is an UNICODE string but realvaluesize is the length in bytes including the leading 0 */
+               *out_value_len=2*(1+biggest_valuesize);
+               *out_data_len=biggest_datasize;
+
+               DEBUG(6,("final values: [%d], [%d]\n", *out_value_len, *out_data_len));
+
+               free_a_printer(printer, 2);             
+               return NT_STATUS_NO_PROBLEMO;
        }
-       else
-       {
-               /* 
-                * the value len is wrong in NT sp3
-                * that's the number of bytes not the number of unicode chars
-                */
-                
-               if (get_specific_param_by_index(printer, 2, idx, value, data, type, &data_len))
-               {
-                       init_unistr(uni_value, value);
-                       
-                       /* the length are in bytes including leading NULL */
-                       (*realvaluesize)=2*(strlen(value)+1);
-                       (*realdatasize)=data_len;
-                       
-                       status=0;
-               }
-               else
-               {
-                       (*valuesize)=0;
-                       (*realvaluesize)=0;
-                       (*datasize)=0;
-                       (*realdatasize)=0;
-                       (*type)=0;
-                       status=0x0103; /* ERROR_NO_MORE_ITEMS */
-               }               
+       
+       /* 
+        * the value len is wrong in NT sp3
+        * that's the number of bytes not the number of unicode chars
+        */
+       if (!get_specific_param_by_index(printer, 2, index, value, &data, &type, &data_len)) {
+               free_a_printer(printer, 2);
+               return 0x0103; /* ERROR_NO_MORE_ITEMS */
        }
+                       
+       /* 
+        * the value is:
+        * - counted in bytes in the request
+        * - counted in UNICODE chars in the max reply
+        * - counted in bytes in the real size
+        *
+        * take a pause *before* coding not *during* coding
+        */
+        
+       *out_max_value_len=in_value_len/2;
+       *out_value=(uint16 *)malloc(in_value_len*sizeof(uint8));
+       ascii_to_unistr(*out_value, value, *out_max_value_len);
+       *out_value_len=2*(1+strlen(value));
+
+       *out_type=type;
+
+       /* the data is counted in bytes */
+       *out_max_data_len=in_data_len;
+       *out_data=(uint8 *)malloc(in_data_len*sizeof(uint8));
+       memcpy(*out_data, data, data_len);
+       *out_data_len=data_len;
+
+       safe_free(data);
        
        free_a_printer(printer, 2);
-
-       return status;
+       return NT_STATUS_NO_PROBLEMO;
 }
 
 /****************************************************************************