mirror of
https://github.com/samba-team/samba.git
synced 2024-12-25 23:21:54 +03:00
Ensure we're not filtering our essential delete messages.
Added jobid debug when unpacking message. Jeremy.
This commit is contained in:
parent
bb60b01f2a
commit
8a6f3313e6
@ -24,6 +24,8 @@
|
||||
|
||||
static TALLOC_CTX *send_ctx;
|
||||
|
||||
static unsigned int num_messages;
|
||||
|
||||
static struct notify_queue {
|
||||
struct notify_queue *next, *prev;
|
||||
struct spoolss_notify_msg *msg;
|
||||
@ -128,6 +130,7 @@ static void print_notify_send_messages_to_printer(const char *printer, unsigned
|
||||
if (!flatten_message(pq)) {
|
||||
DEBUG(0,("print_notify_send_messages: Out of memory\n"));
|
||||
talloc_destroy_pool(send_ctx);
|
||||
num_messages = 0;
|
||||
return;
|
||||
}
|
||||
offset += (pq->buflen + 4);
|
||||
@ -140,6 +143,7 @@ static void print_notify_send_messages_to_printer(const char *printer, unsigned
|
||||
if (!buf) {
|
||||
DEBUG(0,("print_notify_send_messages: Out of memory\n"));
|
||||
talloc_destroy_pool(send_ctx);
|
||||
num_messages = 0;
|
||||
return;
|
||||
}
|
||||
|
||||
@ -190,6 +194,7 @@ void print_notify_send_messages(unsigned int timeout)
|
||||
print_notify_send_messages_to_printer(notify_queue_head->msg->printer, timeout);
|
||||
|
||||
talloc_destroy_pool(send_ctx);
|
||||
num_messages = 0;
|
||||
}
|
||||
|
||||
/**********************************************************************
|
||||
@ -225,11 +230,14 @@ static void send_spoolss_notify2_msg(SPOOLSS_NOTIFY_MSG *msg)
|
||||
struct notify_queue *pnqueue, *tmp_ptr;
|
||||
|
||||
/*
|
||||
* Ensure we only have one message unique to each name/type/field/id/flags
|
||||
* tuple. There is no point in sending multiple messages that match
|
||||
* Ensure we only have one job total_bytes and job total_pages for
|
||||
* each job. There is no point in sending multiple messages that match
|
||||
* as they will just cause flickering updates in the client.
|
||||
*/
|
||||
|
||||
if ((num_messages < 100) && (msg->type == JOB_NOTIFY_TYPE) &&
|
||||
(msg->field == JOB_NOTIFY_TOTAL_BYTES || msg->field == JOB_NOTIFY_TOTAL_PAGES)) {
|
||||
|
||||
for (tmp_ptr = notify_queue_head; tmp_ptr; tmp_ptr = tmp_ptr->next) {
|
||||
if (tmp_ptr->msg->type == msg->type &&
|
||||
tmp_ptr->msg->field == msg->field &&
|
||||
@ -244,6 +252,7 @@ in notify_queue\n", msg->type, msg->field, msg->printer));
|
||||
return;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/* Store the message on the pending queue. */
|
||||
|
||||
@ -274,6 +283,7 @@ to notify_queue_head\n", msg->type, msg->field, msg->printer));
|
||||
*/
|
||||
|
||||
DLIST_ADD_END(notify_queue_head, pnqueue, tmp_ptr);
|
||||
num_messages++;
|
||||
}
|
||||
|
||||
static void send_notify_field_values(const char *printer_name, uint32 type,
|
||||
|
@ -1053,8 +1053,8 @@ static BOOL notify2_unpack_msg( SPOOLSS_NOTIFY_MSG *msg, struct timeval *tv, voi
|
||||
tdb_unpack((char *)buf + offset, len - offset, "B",
|
||||
&msg->len, &msg->notify.data);
|
||||
|
||||
DEBUG(3, ("notify2_unpack_msg: got NOTIFY2 message, type %d, field 0x%02x, flags 0x%04x\n",
|
||||
msg->type, msg->field, msg->flags));
|
||||
DEBUG(3, ("notify2_unpack_msg: got NOTIFY2 message for printer %s, jobid %u type %d, field 0x%02x, flags 0x%04x\n",
|
||||
msg->printer, (unsigned int)msg->id, msg->type, msg->field, msg->flags));
|
||||
|
||||
tv->tv_sec = tv_sec;
|
||||
tv->tv_usec = tv_usec;
|
||||
|
Loading…
Reference in New Issue
Block a user