Skip to content

Commit

Permalink
Fix setting of part name via "name" attribute (#1748)
Browse files Browse the repository at this point in the history
Calling `exr_attr_set_string()` on the "name" attribute sets the part
name. But this was inadvertently setting the part name to the string
"name", not to the actual value of the attribute.

Signed-off-by: Cary Phillips <cary@ilm.com>
  • Loading branch information
cary-ilm committed May 13, 2024
1 parent ef52af0 commit 370db28
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/lib/OpenEXRCore/part_attr.c
Expand Up @@ -1859,7 +1859,7 @@ exr_attr_set_string (
EXR_LOCK_AND_DEFINE_PART (part_index);

if (name && !strcmp (name, EXR_REQ_NAME_STR))
return EXR_UNLOCK_AND_RETURN (exr_set_name (ctxt, part_index, name));
return EXR_UNLOCK_AND_RETURN (exr_set_name (ctxt, part_index, val));

if (name && !strcmp (name, EXR_REQ_TYPE_STR))
return EXR_UNLOCK_AND_RETURN (ctxt->print_error (
Expand Down

0 comments on commit 370db28

Please sign in to comment.