Merge pull request OpenAtomFoundation#368 from Eric-hust/master

[HUST CSE]Modify the null pointer fifo to access first and then judge
AERCR3 · Apr 23, 2023 · 250ca3e · 250ca3e
2 parents 4e17ab6 + 1961644
commit 250ca3e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/components/connectivity/mqttclient/common/log/fifo.c b/components/connectivity/mqttclient/common/log/fifo.c
@@ -98,10 +98,10 @@ unsigned int salof_fifo_read(salof_fifo_t fifo, void *buff, unsigned int len, un
 {
     int l;
 
-    salof_sem_pend(fifo->sem, timeout);
-
     if((!fifo) || (!buff) || (!len))
         return 0;
+
+    salof_sem_pend(fifo->sem, timeout);
 
     len = FIFO_MIN(len, fifo->in - fifo->out);