[Approved] Fix IndexOutOfRangeException in imon lcd driver (1 Viewer)

Waly

MP Donator
  • Premium Supporter
  • July 21, 2009
    18
    3
    Home Country
    Czech Republic Czech Republic
    This patch fixes IndexOutOfRangeException in imon lcd driver. Its because imon font has 255 characters, but when Mediaportal try to show character, which is unicode, it overlaps array bounds. Fix is easy: convert characters to bytes using Encoding.Default. This allows use custom font with right nationals charactes.




    Log from this error

    2009-10-26 23:40:20.146921 [Debug][MiniDisplay]: iMONLCDg.DisplayLines(): CAUGHT EXCEPTION System.IndexOutOfRangeException: Index was outside the bounds of the array.
    v MediaPortal.ProcessPlugins.MiniDisplayPlugin.Drivers.iMONLCDg.SendText(String Line1, String Line2) v D:\Data\Prog\.net\MediaPortal\mediaportal\MiniDisplayLibrary\MiniDisplayPlugin\Drivers\iMONLCDg.cs:line 3151
    v MediaPortal.ProcessPlugins.MiniDisplayPlugin.Drivers.iMONLCDg.DisplayLines() v D:\Data\Prog\.net\MediaPortal\mediaportal\MiniDisplayLibrary\MiniDisplayPlugin\Drivers\iMONLCDg.cs:line 1723
     

    Attachments

    • Fix buffer index overrun_r23998.patch
      30.7 KB

    Users who are viewing this thread

    Top Bottom