From 6392e1b1191211673efb1bf22a697ead77fcb800 Mon Sep 17 00:00:00 2001 From: Edayilliam Jayadev Date: Mon, 4 May 2020 14:54:26 +0530 Subject: [PATCH] qcacmn: Add null pointer check Add a null pointer check after allocating memory for storing the FFT bins. CRs-Fixed: 2673564 Change-Id: I1c55256a1bed921b2272e01a014808b4396bf193 --- target_if/spectral/target_if_spectral_phyerr.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/target_if/spectral/target_if_spectral_phyerr.c b/target_if/spectral/target_if_spectral_phyerr.c index fc8cfa954196..effffba421a7 100644 --- a/target_if/spectral/target_if_spectral_phyerr.c +++ b/target_if/spectral/target_if_spectral_phyerr.c @@ -1430,6 +1430,10 @@ target_if_dump_fft_report_gen3(struct target_if_spectral *spectral, uint32_t *binptr_32 = (uint32_t *)&p_fft_report->buf; fft_bin_buf = (uint8_t *)qdf_mem_malloc(MAX_NUM_BINS); + if (!fft_bin_buf) { + spectral_err("Failed to allocate memory"); + return; + } for (idx = 0; idx < fft_bin_count; idx++) fft_bin_buf[idx] = *(binptr_32++); } else if (spectral->len_adj_swar.fftbin_size_war == @@ -1437,6 +1441,10 @@ target_if_dump_fft_report_gen3(struct target_if_spectral *spectral, uint16_t *binptr_16 = (uint16_t *)&p_fft_report->buf; fft_bin_buf = (uint8_t *)qdf_mem_malloc(MAX_NUM_BINS); + if (!fft_bin_buf) { + spectral_err("Failed to allocate memory"); + return; + } for (idx = 0; idx < fft_bin_count; idx++) fft_bin_buf[idx] = *(binptr_16++); } else {