* alpha: Checking for a failed kmalloc() in process_reloc_for_got()?
@ 2016-12-03 13:15 SF Markus Elfring
2016-12-04 0:10 ` Jiri Kosina
0 siblings, 1 reply; 3+ messages in thread
From: SF Markus Elfring @ 2016-12-03 13:15 UTC (permalink / raw)
To: linux-alpha, Jiri Kosina, Josh Poimboeuf, Matt Turner,
Peter Zijlstra, Richard Henderson, Rusty Russell
Cc: kernel-janitors, linux-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="windows-1254", Size: 644 bytes --]
Hello,
Another run of a small script for the semantic patch language (Coccinelle)
pointed the implementation of the function “process_reloc_for_got†out for
further considerations.
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/arch/alpha/kernel/module.c?idà5f574a0bb1f4502a4b2264fdb0ef6419cf3772#n40
Would it be useful to check the return value from a call of the
function “kmalloc†also there?
Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: alpha: Checking for a failed kmalloc() in process_reloc_for_got()?
2016-12-03 13:15 alpha: Checking for a failed kmalloc() in process_reloc_for_got()? SF Markus Elfring
@ 2016-12-04 0:10 ` Jiri Kosina
2016-12-04 10:20 ` SF Markus Elfring
0 siblings, 1 reply; 3+ messages in thread
From: Jiri Kosina @ 2016-12-04 0:10 UTC (permalink / raw)
To: SF Markus Elfring
Cc: linux-alpha, Josh Poimboeuf, Matt Turner, Peter Zijlstra,
Richard Henderson, Rusty Russell, kernel-janitors, linux-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="windows-1254", Size: 569 bytes --]
On Sat, 3 Dec 2016, SF Markus Elfring wrote:
> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/arch/alpha/kernel/module.c?idà5f574a0bb1f4502a4b2264fdb0ef6419cf3772#n40
>
> Would it be useful to check the return value from a call of the
> function “kmalloc†also there?
*drumroll* it'd actually be a proper bugfix.
--
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: alpha: Checking for a failed kmalloc() in process_reloc_for_got()?
2016-12-04 0:10 ` Jiri Kosina
@ 2016-12-04 10:20 ` SF Markus Elfring
0 siblings, 0 replies; 3+ messages in thread
From: SF Markus Elfring @ 2016-12-04 10:20 UTC (permalink / raw)
To: Jiri Kosina
Cc: linux-alpha, Josh Poimboeuf, Matt Turner, Peter Zijlstra,
Richard Henderson, Rusty Russell, kernel-janitors, linux-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="windows-1254", Size: 765 bytes --]
>> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/arch/alpha/kernel/module.c?idà5f574a0bb1f4502a4b2264fdb0ef6419cf3772#n40
>>
>> Would it be useful to check the return value from a call of the
>> function “kmalloc†also there?
>
> *drumroll* it'd actually be a proper bugfix.
Which kind of exception handling can be added here when the function
“process_reloc_for_got†has got the return type “void†so far?
Will it be needed to handle such a software failure at more source code places
by a special error code?
Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-12-04 10:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-03 13:15 alpha: Checking for a failed kmalloc() in process_reloc_for_got()? SF Markus Elfring
2016-12-04 0:10 ` Jiri Kosina
2016-12-04 10:20 ` SF Markus Elfring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox