-
I've been using NetDump library with ESP8266 & extracted NetDump library from 3.0.0 release.
Here's minimal application:
@earlephilhower @d-a-v @paulocsanz @hreintke |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 3 replies
-
There's type mismatch of printf format vs. the actual type of the argument. diff --git a/libraries/Netdump/src/Netdump.cpp b/libraries/Netdump/src/Netdump.cpp
index 326f2eaa..524a10af 100644
--- a/libraries/Netdump/src/Netdump.cpp
+++ b/libraries/Netdump/src/Netdump.cpp
@@ -140,7 +140,7 @@ void Netdump::writePcapHeader(Stream& s) const
void Netdump::printDumpProcess(Print& out, Packet::PacketDetail ndd, const Packet& np) const
{
- out.printf_P(PSTR("%8d %s"), np.getTime(), np.toString(ndd).c_str());
+ out.printf_P(PSTR("%8lld %s"), np.getTime(), np.toString(ndd).c_str());
}
No warnings from gcc though, since the format string is PSTR? |
Beta Was this translation helpful? Give feedback.
-
The stacktrace also includes
I can confirm that the proposed patch from @mcspr fixes the issue (running on git master). Will check on similar statements in the code and submit a PR. |
Beta Was this translation helpful? Give feedback.
-
Fixed by @d-a-v in 3.0.2. See defect #8215 for more details. |
Beta Was this translation helpful? Give feedback.
Fixed by @d-a-v in 3.0.2. See defect #8215 for more details.