Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 56ad28bf authored by Edayilliam Jayadev's avatar Edayilliam Jayadev Committed by nshrivas
Browse files

qcacmn: Add null pointer check 

Add a null pointer check after allocating memory for storing the
FFT bins.

CRs-Fixed: 2673564
Change-Id: I1c55256a1bed921b2272e01a014808b4396bf193
parent f03bcd6b

target_if/spectral/target_if_spectral_phyerr.c

+8 −0
Original line number Diff line number Diff line
@@ -1392,6 +1392,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 ==
@@ -1399,6 +1403,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 {