Fix copilot feedback and address remaining issues

Author: aidangarske

src/mqtt_broker.c

@@ -1025,7 +1025,7 @@ static int BrokerWsNetWrite(void* context, const byte* buf, int buf_len,
 
     /* Check if the write callback reported an error (it frees tx_pending
      * and sets status to -1 before returning -1 to lws) */
-    if (ws->status <= 0) {
+    if (ws->status < 0) {
         return MQTT_CODE_ERROR_NETWORK;
     }
 
@@ -3867,7 +3867,8 @@ int MqttBroker_Start(MqttBroker* broker)
             broker->port != broker->port_tls) {
             WBLOG_ERR(broker,
                 "broker: WARNING: auth credentials exposed on plaintext "
-                "port %d. Use --disable-broker-insecure for TLS-only",
+                "port %d. Rebuild with ./configure --disable-broker-insecure "
+                "for TLS-only",
                 broker->port);
         }
     #endif

wolfmqtt/mqtt_types.h

@@ -362,8 +362,11 @@ enum MqttPacketResponseCodes {
  * from optimizing away the stores (dead-store elimination). */
 static INLINE void Mqtt_ForceZero(void* mem, word32 len)
 {
-    volatile byte* z = (volatile byte*)mem;
-    while (len--) *z++ = 0;
+    volatile byte* p = (volatile byte*)mem;
+    word32 i;
+    for (i = 0; i < len; i++) {
+        p[i] = 0;
+    }
 }
 
 /* Logging / Tracing */