Author: gd Date: 2005-03-18 21:35:43 +0000 (Fri, 18 Mar 2005) New Revision: 5886
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=5886 Log: Minor fixes for printerdb. Allows very basic use of the file-backend. Guenther Modified: trunk/source/printing/printerdb.c trunk/source/printing/printerdb_file.c Changeset: Modified: trunk/source/printing/printerdb.c =================================================================== --- trunk/source/printing/printerdb.c 2005-03-18 16:05:05 UTC (rev 5885) +++ trunk/source/printing/printerdb.c 2005-03-18 21:35:43 UTC (rev 5886) @@ -154,15 +154,22 @@ static BOOL printerdb_valid(struct printerdb_methods *printerdb, int tdb) { - time_t t = time(NULL); + time_t t; + if (lp_printerdb_cache_time() <= 0) { + DEBUG(10,("printerdb_valid: tdb caching disabled\n")); + return False; + } + + t = time(NULL); + if (t < printerdb->get_last_update(tdb) + lp_printerdb_cache_time()) { DEBUG(10,("printerdb_valid: printerdb still valid (%d < %d)\n", (int)t, (int)printerdb->get_last_update(tdb) + lp_printerdb_cache_time())); return True; } - DEBUG(3,("printerdb_valid: cache expired, asking remote backend\n")); + DEBUG(10,("printerdb_valid: cache expired, asking remote backend\n")); return False; } Modified: trunk/source/printing/printerdb_file.c =================================================================== --- trunk/source/printing/printerdb_file.c 2005-03-18 16:05:05 UTC (rev 5885) +++ trunk/source/printing/printerdb_file.c 2005-03-18 21:35:43 UTC (rev 5886) @@ -84,7 +84,7 @@ if (*num_files == 0) return False; - DEBUG(0,("found: %d files\n", *num_files)); + DEBUG(10,("found: %d files\n", *num_files)); return True; } @@ -330,7 +330,7 @@ if (ret != len) goto done; - DEBUG(0,("file_get_c_setprinter: will return %d\n", result)); + DEBUG(10,("file_get_c_setprinter: will return %d\n", result)); done: if (mem_ctx != NULL) @@ -817,7 +817,7 @@ result = sec_io_desc_buf("file_get_secdesc", secdesc_ctr, &ps, 1) ? WERR_OK : WERR_NOMEM; - prs_mem_free(&ps); + /* prs_mem_free(&ps); */ return result; } @@ -959,12 +959,6 @@ len += unpack_values( &info.data, buf+len, buflen-len ); - if (len != buflen) { - DEBUG(0,("len: %d != buflen %d\n", len, buflen)); - result = WERR_INVALID_PARAM; - goto done; - } - *info_ptr = (NT_PRINTER_INFO_LEVEL_2 *)memdup(&info, sizeof(info)); done: