From: Pavel Shilovsky Date: Mon, 28 May 2012 11:50:10 +0000 (+0400) Subject: CIFS: Make accessing is_valid_oplock/dump_detail ops struct field safe X-Git-Tag: v3.5-rc2~36^2~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7f0adb53bcf5bdb92236cda8ec92ea5e40993028;p=profile%2Fivi%2Fkernel-adaptation-intel-automotive.git CIFS: Make accessing is_valid_oplock/dump_detail ops struct field safe Signed-off-by: Pavel Shilovsky Signed-off-by: Steve French --- diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index ccafded..3647b1a 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -1058,13 +1058,15 @@ cifs_demultiplex_thread(void *p) if (mid_entry != NULL) { if (!mid_entry->multiRsp || mid_entry->multiEnd) mid_entry->callback(mid_entry); - } else if (!server->ops->is_oplock_break(buf, server)) { + } else if (!server->ops->is_oplock_break || + !server->ops->is_oplock_break(buf, server)) { cERROR(1, "No task to wake, unknown frame received! " "NumMids %d", atomic_read(&midCount)); cifs_dump_mem("Received Data is: ", buf, HEADER_SIZE(server)); #ifdef CONFIG_CIFS_DEBUG2 - server->ops->dump_detail(buf); + if (server->ops->dump_detail) + server->ops->dump_detail(buf); cifs_dump_mids(server); #endif /* CIFS_DEBUG2 */