#15 makeutype.c update

closed-accepted
nobody
None
5
2012-07-26
2012-07-26
No

Fixed some possible buffer[] overruns.

Discussion

  • Jose Da Silva

    Jose Da Silva - 2012-07-26

    makeutype.c update and fixes

     
    Attachments
  • Jose Da Silva

    Jose Da Silva - 2012-07-26

    makeutype.c additional update (forgot to include this)

     
    Attachments
  • Michał Nowakowski

    Thanks, applied.

     
  • Michał Nowakowski

    • status: open --> closed-accepted
     
  • Michał Nowakowski

    I had to adjust the patches slightly, as cppcheck complained:

    [makeutype.c:202]: (error) Array 'buffer[301]' index 301 out of bounds
    [makeutype.c:202]: (error) Array 'buf2[301]' index 301 out of bounds
    [makeutype.c:475]: (error) Array 'buffer[301]' index 301 out of bounds
    [makeutype.c:475]: (error) Array 'buf2[301]' index 301 out of bounds
    [makeutype.c:512]: (error) Array 'name[301]' index 301 out of bounds

     
  • Jose Da Silva

    Jose Da Silva - 2012-07-30

    Thanks Michal for looking-out for these things.
    Since you've already included/accepted this patch, I started a new patch at #3551379

    In future, do you want these fixes submitted back to the same patch, or to start a new patch?

    Thanks,
    Joe

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks