util: Make out_of_memory() call abort() instead of exit(EXIT_FAILURE).
authorBen Pfaff <blp@nicira.com>
Wed, 23 Feb 2011 23:43:34 +0000 (15:43 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 23 Feb 2011 23:43:34 +0000 (15:43 -0800)
commitc1c8308a3971526f5126201a6d1659c9247c1f0a
tree68adab7c1d6b6cd275ff3236d1029c443aa47910
parent1883ed0f727bd10f0b8b6b80832029c349138595
util: Make out_of_memory() call abort() instead of exit(EXIT_FAILURE).

exit(EXIT_FAILURE) will make a monitoring process (the one created by
--monitor) think that it should exit.  But the most likely reason for
out_of_memory() to be called is a bug: probably, the process is trying
to allocate more memory than there is available address space, e.g.
something like malloc(-1).  So it's better, in my opinion, to call abort()
instead, so that the monitor process restarts the daemon and we are more
likely to stay alive and, in addition, get a core dump and a useful bug
report.

I decided to implement a new general-purpose function for this purpose in
case we run into other similar situations in the future.

(I haven't actually run into this problem in practice.  This commit is
just speculation about what is better behavior.)
lib/util.c
lib/util.h