psdump icon indicating copy to clipboard operation
psdump copied to clipboard

fix/psd_parser_deep_nesting

Open irov opened this issue 8 years ago • 2 comments

Section divider setting sometime has 16 byte code length

code to fix this bug


    if(size >= 12)
    {
        // Signature: '8BIM'
        if(psd_stream_get_int(context) != '8BIM')
            return psd_status_divider_signature_error;

        // blend mode
        layer->divider_blend_mode = psd_stream_get_blend_mode(context);
    }

    // Following is only present if length == 16
    if( size == 16 )
    {
        //Sub type. 0 = normal, 1 = scene group, affects the animation timeline.
        int divider_animation_timeline_mode = psd_stream_get_int( context );

        layer->divider_animation_timeline_mode = divider_animation_timeline_mode;
    }

irov avatar Jul 28 '16 08:07 irov

Hi @irov. Could please clarify which source file this code is in? Also, do you have a sample PSD file that reproduces the issue?

alco avatar Jul 30 '16 09:07 alco

Hi, source in file layer_mask.c line 265 PSD is simple, you must create 10+ sub group (maybe enough 8) like this -Group 1 --Group 2 ---Group 3 ----Group 4 -----Group 5 ------Group 6 ---------etc ----------Layer 1

P.S. I create on next week new fix & issues)) Coming soon

irov avatar Jul 30 '16 15:07 irov