Minigotchi crashing with Config::advertise = true; about minigotchi-esp32 HOT 8 CLOSED

@ATOMNFT and I just recently solved the issue, if you want to try compiling from the development branch for the ESP32.

from minigotchi-esp32.

I know the issue is being worked on, but I haven't found much off of exception decoders. The known workaround is to just set it to false.

from minigotchi-esp32.

Sorry for opening up this issue, did not read the the comment on minigotchi.

from minigotchi-esp32.

nah it's all good. I'm in the progress of fixing it but I haven't really close to fixing the exception

from minigotchi-esp32.

I think this is one problematic area. Instead of appending each byte from Frame::beaconFrame, it should be appending data from Frame::payload.
The pack() function constructs the entire beacon frame by first initializing it (init()), appending ESSID data (essid()), and then chunking the data into packets using chunkSize. There's a logical error in how the payload is being copied

from minigotchi-esp32.

I think this is one problematic area. Instead of appending each byte from Frame::beaconFrame, it should be appending data from Frame::payload.
The pack() function constructs the entire beacon frame by first initializing it (init()), appending ESSID data (essid()), and then chunking the data into packets using chunkSize. There's a logical error in how the payload is being copied

I could be totally wrong though...

from minigotchi-esp32.

I think this is one problematic area. Instead of appending each byte from Frame::beaconFrame, it should be appending data from Frame::payload. The pack() function constructs the entire beacon frame by first initializing it (init()), appending ESSID data (essid()), and then chunking the data into packets using chunkSize. There's a logical error in how the payload is being copied

Frame::payload doesn't exist. there's Frame::payloadSize which is used to measure the length of the essid + header without the chunking.

/**
 * frame.cpp
 */

void Frame::pack() {
  // clear frame before constructing
  Frame::beaconFrame.clear();

  // add the header and essid
  init();
  essid();

  // payload size
  Frame::payloadSize = Frame::beaconFrame.size();

  // store for later
  std::vector<uint8_t> originalBeaconFrame = Frame::beaconFrame;

  for (size_t i = 0; i < payloadSize; i += Frame::chunkSize) {
    Frame::beaconFrame.push_back(Frame::IDWhisperPayload);

    size_t chunkEnd = std::min(i + Frame::chunkSize, Frame::payloadSize);
    Frame::beaconFrame.push_back(chunkEnd - i);

    for (size_t j = i; j < chunkEnd; ++j) {
      Frame::beaconFrame.push_back(originalBeaconFrame[j]);
    }
  }

  // save some memory
  originalBeaconFrame.clear();
  originalBeaconFrame.shrink_to_fit();

  // update size since we changed the frame
  Frame::frameSize = Frame::beaconFrame.size();

}

When constructing the frame there are a couple of steps that need to be done

1. Header (appended to Frame::beaconFrame)
2. JSON Data (appended to Frame::beaconFrame)
3. For every 255 bytes we will need to chunk the payload and throw in something called Frame::IDWhisperPayload to indicate this is from a Pwnagotchi

Something I've noticed with ESP32 development is that the compiler is a lot more strict when it comes to declaring objects, function signatures, etc. I'll probably need to adjust accordingly. I'll read up more on pack.go from pwngrid if needed

from minigotchi-esp32.

Going to close this as advertising works now.

from minigotchi-esp32.