Re: [PATCH V1] rpmsg: glink: Initialize the "intent_req_comp" completion variable
On Sun 29 Oct 22:41 PDT 2017, Arun Kumar Neelakantam wrote: > The "intent_req_comp" variable is used without initialization which > results in NULL pointer dereference in qcom_glink_request_intent(). > > we need to initialize the completion variable before using it. > > Fixes: 27b9c5b66b23 ("rpmsg: glink: Request for intents when unavailable") > Signed-off-by: Arun Kumar NeelakantamThanks, applied. Regards, Bjorn > --- > drivers/rpmsg/qcom_glink_native.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/rpmsg/qcom_glink_native.c > b/drivers/rpmsg/qcom_glink_native.c > index 5dcc9bf..fcd46ab 100644 > --- a/drivers/rpmsg/qcom_glink_native.c > +++ b/drivers/rpmsg/qcom_glink_native.c > @@ -227,6 +227,7 @@ static struct glink_channel > *qcom_glink_alloc_channel(struct qcom_glink *glink, > > init_completion(>open_req); > init_completion(>open_ack); > + init_completion(>intent_req_comp); > > INIT_LIST_HEAD(>done_intents); > INIT_WORK(>intent_work, qcom_glink_rx_done_work); > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project >
Re: [PATCH V1] rpmsg: glink: Initialize the "intent_req_comp" completion variable
On Sun 29 Oct 22:41 PDT 2017, Arun Kumar Neelakantam wrote: > The "intent_req_comp" variable is used without initialization which > results in NULL pointer dereference in qcom_glink_request_intent(). > > we need to initialize the completion variable before using it. > > Fixes: 27b9c5b66b23 ("rpmsg: glink: Request for intents when unavailable") > Signed-off-by: Arun Kumar Neelakantam Thanks, applied. Regards, Bjorn > --- > drivers/rpmsg/qcom_glink_native.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/rpmsg/qcom_glink_native.c > b/drivers/rpmsg/qcom_glink_native.c > index 5dcc9bf..fcd46ab 100644 > --- a/drivers/rpmsg/qcom_glink_native.c > +++ b/drivers/rpmsg/qcom_glink_native.c > @@ -227,6 +227,7 @@ static struct glink_channel > *qcom_glink_alloc_channel(struct qcom_glink *glink, > > init_completion(>open_req); > init_completion(>open_ack); > + init_completion(>intent_req_comp); > > INIT_LIST_HEAD(>done_intents); > INIT_WORK(>intent_work, qcom_glink_rx_done_work); > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project >
[PATCH V1] rpmsg: glink: Initialize the "intent_req_comp" completion variable
The "intent_req_comp" variable is used without initialization which results in NULL pointer dereference in qcom_glink_request_intent(). we need to initialize the completion variable before using it. Fixes: 27b9c5b66b23 ("rpmsg: glink: Request for intents when unavailable") Signed-off-by: Arun Kumar Neelakantam--- drivers/rpmsg/qcom_glink_native.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rpmsg/qcom_glink_native.c b/drivers/rpmsg/qcom_glink_native.c index 5dcc9bf..fcd46ab 100644 --- a/drivers/rpmsg/qcom_glink_native.c +++ b/drivers/rpmsg/qcom_glink_native.c @@ -227,6 +227,7 @@ static struct glink_channel *qcom_glink_alloc_channel(struct qcom_glink *glink, init_completion(>open_req); init_completion(>open_ack); + init_completion(>intent_req_comp); INIT_LIST_HEAD(>done_intents); INIT_WORK(>intent_work, qcom_glink_rx_done_work); -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
[PATCH V1] rpmsg: glink: Initialize the "intent_req_comp" completion variable
The "intent_req_comp" variable is used without initialization which results in NULL pointer dereference in qcom_glink_request_intent(). we need to initialize the completion variable before using it. Fixes: 27b9c5b66b23 ("rpmsg: glink: Request for intents when unavailable") Signed-off-by: Arun Kumar Neelakantam --- drivers/rpmsg/qcom_glink_native.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rpmsg/qcom_glink_native.c b/drivers/rpmsg/qcom_glink_native.c index 5dcc9bf..fcd46ab 100644 --- a/drivers/rpmsg/qcom_glink_native.c +++ b/drivers/rpmsg/qcom_glink_native.c @@ -227,6 +227,7 @@ static struct glink_channel *qcom_glink_alloc_channel(struct qcom_glink *glink, init_completion(>open_req); init_completion(>open_ack); + init_completion(>intent_req_comp); INIT_LIST_HEAD(>done_intents); INIT_WORK(>intent_work, qcom_glink_rx_done_work); -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project