bcm accelerated blit: avoid running out of clut data

We were passing a hardcoded number of 256 entries to the framebuffer,
while the source surface could have a smaller color lookup table.
For instance, most dvb subtitle graphics have a palette of only 16 colors.
This would cause frequent segfaults.

diff --git a/lib/gdi/accel.cpp b/lib/gdi/accel.cpp
index 9450eccaf400d8c773627d7da4ebc3f2fd8afb13..fc739e92b917e3058b4bbb3f360a94a8b7723b66 100644 (file)
--- a/lib/gdi/accel.cpp
+++ b/lib/gdi/accel.cpp
@@ -112,7 +112,7 @@ int gAccel::blit(gSurface *dst, const gSurface *src, const eRect &p, const eRect
                        pal_addr = src->stride * src->y;
                        unsigned long *pal = (unsigned long*)(((unsigned char*)src->data) + pal_addr);
                        pal_addr += src->data_phys;
                        pal_addr = src->stride * src->y;
                        unsigned long *pal = (unsigned long*)(((unsigned char*)src->data) + pal_addr);
                        pal_addr += src->data_phys;

-                       for (i = 0; i < 256; ++i)
+                       for (i = 0; i < src->clut.colors; ++i)

                                *pal++ = src->clut.data[i].argb() ^ 0xFF000000;
                } else
                        return -1; /* unsupported source format */
                                *pal++ = src->clut.data[i].argb() ^ 0xFF000000;
                } else
                        return -1; /* unsupported source format */