Re: [PATCH] vsh: Don't check for OOM in vshGetTypedParamValue()

On a Friday in 2021, Michal Privoznik wrote:

Both function description and function itself mention check for OOM which can't happen really. There was a bug in glib where g_strdup_*() might have not aborted on OOM, but we have our own implementation when dealing with broken glib (see vir_g_strdup_printf()). Therefore, checking for OOM is redundant and can never be true.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com> --- tools/vsh.c | 25 +++++++++---------------- 1 file changed, 9 insertions(+), 16 deletions(-)

Reviewed-by: Ján Tomko <jtomko@redhat.com> Jano