Commit fd1c86f4 authored by Arun Gopalakrishnan's avatar Arun Gopalakrishnan Committed by Hari Kanigeri

ARM OMAP4 Syslink IPC gatepeterson, nameserver remote notify fix

This fix includes
[gatepeterson] A check for NULL for a parameter (name), that was not
necessary, was removed.

[nameserver remote notify] The nameserver instance handle was got
correctly from nameserver module but not used it. Instead we passed
an uninitialized variable to next nameserver call, that made a BUGON
Signed-off-by: default avatarArun M G <arunmg@ti.com>
parent 7a7911fd
...@@ -281,11 +281,6 @@ void *gatepeterson_create(const struct gatepeterson_params *params) ...@@ -281,11 +281,6 @@ void *gatepeterson_create(const struct gatepeterson_params *params)
goto exit; goto exit;
} }
if (WARN_ON(params->name == NULL)) {
retval = -EINVAL;
goto exit;
}
handle = kmalloc(sizeof(struct gatepeterson_object), GFP_KERNEL); handle = kmalloc(sizeof(struct gatepeterson_object), GFP_KERNEL);
if (handle == NULL) { if (handle == NULL) {
retval = -ENOMEM; retval = -ENOMEM;
......
...@@ -278,7 +278,6 @@ void nameserver_remotenotify_callback(u16 proc_id, u32 event_no, ...@@ -278,7 +278,6 @@ void nameserver_remotenotify_callback(u16 proc_id, u32 event_no,
u32 value; u32 value;
u32 key; u32 key;
s32 retval; s32 retval;
void *entry = NULL;
BUG_ON(arg == NULL); BUG_ON(arg == NULL);
proc_count = multiproc_get_max_processors(); proc_count = multiproc_get_max_processors();
...@@ -293,8 +292,9 @@ void nameserver_remotenotify_callback(u16 proc_id, u32 event_no, ...@@ -293,8 +292,9 @@ void nameserver_remotenotify_callback(u16 proc_id, u32 event_no,
goto exit; goto exit;
/* This is a request */ /* This is a request */
entry = nameserver_get_handle(handle->msg[1 - offset]->instance_name); nshandle = nameserver_get_handle(
if (entry == NULL) handle->msg[1 - offset]->instance_name);
if (nshandle == NULL)
goto exit; goto exit;
/* Search for the NameServer entry */ /* Search for the NameServer entry */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment