Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#15 makeutype.c update

closed-accepted
nobody
None
5
2012-07-26
2012-07-26
Jose Da Silva
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
  • Thanks, applied.

     
    • status: open --> closed-accepted
     
  • 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