Alex Dewar <alex.dewar90@xxxxxxxxx> wrote: > The for loop checks whether cur_section is NULL on every iteration, but > we know it can never be NULL as there is another check towards the > bottom of the loop body. Refactor to avoid this unnecessary check. > > Also, increment the variable i inline for clarity > > Addresses-Coverity: 1496984 ("Null pointer dereferences) > Suggested-by: Saeed Mahameed <saeedm@xxxxxxxxxx> > Signed-off-by: Alex Dewar <alex.dewar90@xxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. dbeb101d28eb ath10k: sdio: remove redundant check in for loop -- https://patchwork.kernel.org/project/linux-wireless/patch/20200916165748.20927-1-alex.dewar90@xxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches