summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngie Chiang <angiebird@google.com>2018-11-19 10:25:35 -0800
committerAngie Chiang <angiebird@google.com>2018-11-19 18:45:23 -0800
commit0310ebd8d1f44eca865f33c063ad4410969d3b94 (patch)
tree04b71ba37347c39ccfbeb55cd11a023220c06a64
parent50bbc0984c736133830ecdeb710b003dab2af699 (diff)
downloadlibvpx-0310ebd8d1f44eca865f33c063ad4410969d3b94.tar
libvpx-0310ebd8d1f44eca865f33c063ad4410969d3b94.tar.gz
libvpx-0310ebd8d1f44eca865f33c063ad4410969d3b94.tar.bz2
libvpx-0310ebd8d1f44eca865f33c063ad4410969d3b94.zip
Fix scan_build warnings in tiny_ssim.c
BUG=webm:1575 Change-Id: I3ad3af49d778f102e9152dcb1eb9d5c048756cdf
-rw-r--r--tools/tiny_ssim.c12
1 files changed, 12 insertions, 0 deletions
diff --git a/tools/tiny_ssim.c b/tools/tiny_ssim.c
index 6f4b6d735..75ef569ce 100644
--- a/tools/tiny_ssim.c
+++ b/tools/tiny_ssim.c
@@ -53,6 +53,10 @@ static uint64_t calc_plane_error(uint8_t *orig, int orig_stride, uint8_t *recon,
unsigned int row, col;
uint64_t total_sse = 0;
int diff;
+ if (orig == NULL || recon == NULL) {
+ assert(0);
+ return 0;
+ }
for (row = 0; row < rows; row++) {
for (col = 0; col < cols; col++) {
@@ -99,6 +103,9 @@ static int open_input_file(const char *file_name, input_file_t *input, int w,
int h, int bit_depth) {
char y4m_buf[4];
size_t r1;
+ input->w = w;
+ input->h = h;
+ input->bit_depth = bit_depth;
input->type = RAW_YUV;
input->buf = NULL;
input->file = strcmp(file_name, "-") ? fopen(file_name, "rb") : stdin;
@@ -187,6 +194,11 @@ void ssim_parms_8x8(const uint8_t *s, int sp, const uint8_t *r, int rp,
uint32_t *sum_s, uint32_t *sum_r, uint32_t *sum_sq_s,
uint32_t *sum_sq_r, uint32_t *sum_sxr) {
int i, j;
+ if (s == NULL || r == NULL || sum_s == NULL || sum_r == NULL ||
+ sum_sq_s == NULL || sum_sq_r || sum_sxr == NULL) {
+ assert(0);
+ return;
+ }
for (i = 0; i < 8; i++, s += sp, r += rp) {
for (j = 0; j < 8; j++) {
*sum_s += s[j];