W dniu 27.06.2017 o 18:08, Thierry Escande pisze:
This patch modifies the s5p_jpeg_parse_hdr() function so it only
modifies the passed s5p_jpeg_q_data structure if the jpeg header parsing
is successful.

Signed-off-by: Thierry Escande <thierry.esca...@collabora.com>

Acked-by: Andrzej Pietrasiewicz <andrze...@samsung.com>

---
  drivers/media/platform/s5p-jpeg/jpeg-core.c | 38 ++++++++++++++++-------------
  1 file changed, 21 insertions(+), 17 deletions(-)

diff --git a/drivers/media/platform/s5p-jpeg/jpeg-core.c 
b/drivers/media/platform/s5p-jpeg/jpeg-core.c
index 0d935f5..df3e5ee 100644
--- a/drivers/media/platform/s5p-jpeg/jpeg-core.c
+++ b/drivers/media/platform/s5p-jpeg/jpeg-core.c
@@ -1206,22 +1206,9 @@ static bool s5p_jpeg_parse_hdr(struct s5p_jpeg_q_data 
*result,
                        break;
                }
        }
-       result->w = width;
-       result->h = height;
-       result->sos = sos;
-       result->dht.n = n_dht;
-       while (n_dht--) {
-               result->dht.marker[n_dht] = dht[n_dht];
-               result->dht.len[n_dht] = dht_len[n_dht];
-       }
-       result->dqt.n = n_dqt;
-       while (n_dqt--) {
-               result->dqt.marker[n_dqt] = dqt[n_dqt];
-               result->dqt.len[n_dqt] = dqt_len[n_dqt];
-       }
-       result->sof = sof;
-       result->sof_len = sof_len;
-       result->size = result->components = components;
+
+       if (notfound || !sos)
+               return false;
switch (subsampling) {
        case 0x11:
@@ -1240,7 +1227,24 @@ static bool s5p_jpeg_parse_hdr(struct s5p_jpeg_q_data 
*result,
                return false;
        }
- return !notfound && sos;
+       result->w = width;
+       result->h = height;
+       result->sos = sos;
+       result->dht.n = n_dht;
+       while (n_dht--) {
+               result->dht.marker[n_dht] = dht[n_dht];
+               result->dht.len[n_dht] = dht_len[n_dht];
+       }
+       result->dqt.n = n_dqt;
+       while (n_dqt--) {
+               result->dqt.marker[n_dqt] = dqt[n_dqt];
+               result->dqt.len[n_dqt] = dqt_len[n_dqt];
+       }
+       result->sof = sof;
+       result->sof_len = sof_len;
+       result->size = result->components = components;
+
+       return true;
  }
static int s5p_jpeg_querycap(struct file *file, void *priv,


Reply via email to