depmod: abort on OOM
authorAlan Jenkins <alan-jenkins@tuffmail.co.uk>
Tue, 10 Feb 2009 16:01:10 +0000 (16:01 +0000)
committerKay Sievers <kay.sievers@vrfy.org>
Tue, 10 Feb 2009 16:23:36 +0000 (17:23 +0100)
commit8037146631f4e3c870eed476b71e278b7c136201
tree70ca356de87bffd9d2127a6632e99090a947dfd2
parent641c8b13c05189ef58b596de45851636ece5084c
depmod: abort on OOM

Alan Jenkins napsal(a):
> Michal Marek wrote:
> > if depmod runs out memory reading modules, it will print a "WARNING:
> > Can't read module ....ko" line for each module it fails to load and
> > later, depending on the moon phase, either abort when a malloc() or
> > strdup() fails, or it will simply miss dependencies and exit
> > successfully. Attached patch makes depmod abort in case of OOM (but
> > doesn't abort if you have e.g. stale symlinks in the modules tree).

Wouldn't it be better to move zlibsupport.c into line with the rest of
m-i-t, and abort immediately on allocation failures?  It seems a better
way forward now we have a shared "logging.h" (since version 3.5).
zlibsupport.c