Menu

#144 GeneralConvolution + unaligned data caus access violation

v2.5x
closed-fixed
Filters (75)
5
2005-05-31
2005-05-11
Anonymous
No

When applying GeneralConvolution to a RGBA-Clip that
has been cropped (without align=true) previously, an
access violation is reported.

I tried the following script in Win2k SP4 / AviSynth
2.56 beta3 20050505 / VirtualDub 1.5.10 b.18160. No
other AviSynth plugins are loaded.
###########################################
ColorBars
Crop(5,5,-5,-5)
GeneralConvolution(0,"1 2 3 4 5 6 7 8 9")
###########################################
Opening in VirtualDub makes VirtualDub display
"Avisynth read error: Avisynth: caught an access
violation at 0x1003cbaf, attempting to write to
0x018e3000" (Maybe those numbers are helpfull when
debugging). Sometimes (but I couldn't figure out when),
the first frame is displayed (a "colorfull"
modification of the frame it should be), but hitting
the play button give a dialog box with "...access
violation...".

Everything works fine when adding "align=true" to the
Crop-command.

BTW: There seems to be a lack of documentation: How are
border pixels treated by GeneralConvolution? (Three
obvious answers: black border, last pixel line is
continues, image is mirrored)

I hope I helped you in developing AviSynth.
Sorry for my crappy English.

Discussion

  • Ian Brabham

    Ian Brabham - 2005-05-30
    • labels: --> Filters
    • milestone: --> v2.5x
    • assigned_to: nobody --> ianb1957
    • status: open --> pending-fixed
     
  • Ian Brabham

    Ian Brabham - 2005-05-30

    Logged In: YES
    user_id=673887

    Ah the evils of assuming pitch. Code assumes input and
    output pitch are the same. This is wrong period! It will be fixed
    in the next release.

    Borders : first/last pixel/line is repeated. DOcs will be
    updated.

    IanB

     
  • Ian Brabham

    Ian Brabham - 2005-05-31
    • status: pending-fixed --> closed-fixed
     
  • Ian Brabham

    Ian Brabham - 2005-05-31

    Logged In: YES
    user_id=673887

    Commited fix to CVS, available next build

    IanB

     

Log in to post a comment.

MongoDB Logo MongoDB