Taking into respect the return value of a function which can fail.

Coverity CID 12595.
This commit is contained in:
Thomas Schmitt 2015-10-11 10:15:38 +02:00
parent a4c1e04820
commit 7c29a94ab6

View File

@ -1,6 +1,6 @@
/* /*
* Copyright (c) 2007 Vreixo Formoso * Copyright (c) 2007 Vreixo Formoso
* Copyright (c) 2009 - 2014 Thomas Schmitt * Copyright (c) 2009 - 2015 Thomas Schmitt
* *
* This file is part of the libisofs project; you can redistribute it and/or * This file is part of the libisofs project; you can redistribute it and/or
* modify it under the terms of the GNU General Public License version 2 * modify it under the terms of the GNU General Public License version 2
@ -233,9 +233,9 @@ int default_create_node(IsoNodeBuilder *builder, IsoImage *image,
/* Eventually set S_IRWXG from ACL */ /* Eventually set S_IRWXG from ACL */
if (image->builder_ignore_acl) { if (image->builder_ignore_acl) {
ret = iso_file_source_get_aa_string(src, &aa_string, 4); ret = iso_file_source_get_aa_string(src, &aa_string, 4);
if (aa_string != NULL) if (ret >= 0 && aa_string != NULL)
iso_aa_get_acl_text(aa_string, info.st_mode, &a_text, &d_text, 16); iso_aa_get_acl_text(aa_string, info.st_mode, &a_text, &d_text, 16);
if (a_text != NULL) { if (ret >= 0 && a_text != NULL) {
aaip_cleanout_st_mode(a_text, &(info.st_mode), 4 | 16); aaip_cleanout_st_mode(a_text, &(info.st_mode), 4 | 16);
iso_node_set_perms_internal(new, info.st_mode, 1); iso_node_set_perms_internal(new, info.st_mode, 1);
} }