MarcoBuster/Gadgetbridge
1
0
Fork 0
mirror of https://codeberg.org/Freeyourgadget/Gadgetbridge synced 2024-11-15 14:39:26 +01:00
Code Releases Wiki Activity

Garmin protocol: store max packet size from DeviceInformationMessage

Browse Source 
also adds messageType to the warnifleftover log message
This commit is contained in:
Daniele Gobbetti 2024-04-07 19:50:18 +02:00
parent 6cb5184e2b
commit 14c6ce1a3a
2 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/garmin/messages/DeviceInformationMessage.java
View File

@ -35,6 +35,7 @@ public class DeviceInformationMessage extends GFDIMessage {
this.deviceName = deviceName; this.deviceName = deviceName;
this.deviceModel = deviceModel; this.deviceModel = deviceModel;
GFDIMessage.setMaxPacketSize(maxPacketSize);
this.statusMessage = getStatusMessage(); this.statusMessage = getStatusMessage();
} }

15
app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/garmin/messages/GFDIMessage.java
View File

@ -32,10 +32,19 @@ public abstract class GFDIMessage {
public static final int MESSAGE_SYNC_REQUEST = 5037; public static final int MESSAGE_SYNC_REQUEST = 5037;
public static final int MESSAGE_AUTH_NEGOTIATION = 5101; public static final int MESSAGE_AUTH_NEGOTIATION = 5101;
protected static final Logger LOG = LoggerFactory.getLogger(GFDIMessage.class); protected static final Logger LOG = LoggerFactory.getLogger(GFDIMessage.class);
private static int maxPacketSize = 375; //safe default?
protected final ByteBuffer response = ByteBuffer.allocate(1000); protected final ByteBuffer response = ByteBuffer.allocate(1000);
protected GFDIStatusMessage statusMessage; protected GFDIStatusMessage statusMessage;
protected GarminMessage garminMessage; protected GarminMessage garminMessage;
public static int getMaxPacketSize() {
return maxPacketSize;
}
public static void setMaxPacketSize(int maxPacketSize) {
GFDIMessage.maxPacketSize = maxPacketSize;
}
public static GFDIMessage parseIncoming(byte[] message) { public static GFDIMessage parseIncoming(byte[] message) {
final MessageReader messageReader = new MessageReader(message); final MessageReader messageReader = new MessageReader(message);
@ -48,7 +57,7 @@ public abstract class GFDIMessage {
LOG.error("UNHANDLED GFDI MESSAGE TYPE {}, MESSAGE {}", messageType, message); LOG.error("UNHANDLED GFDI MESSAGE TYPE {}, MESSAGE {}", messageType, message);
return new UnhandledMessage(messageType); return new UnhandledMessage(messageType);
} finally { } finally {
messageReader.warnIfLeftover(); messageReader.warnIfLeftover(messageType);
} }
} }
@ -194,14 +203,14 @@ public abstract class GFDIMessage {
} }
} }
public void warnIfLeftover() { public void warnIfLeftover(int messageType) {
if (byteBuffer.hasRemaining() && byteBuffer.position() < (byteBuffer.limit())) { if (byteBuffer.hasRemaining() && byteBuffer.position() < (byteBuffer.limit())) {
int pos = byteBuffer.position(); int pos = byteBuffer.position();
int numBytes = (byteBuffer.limit()) - byteBuffer.position(); int numBytes = (byteBuffer.limit()) - byteBuffer.position();
byte[] leftover = new byte[numBytes]; byte[] leftover = new byte[numBytes];
byteBuffer.get(leftover); byteBuffer.get(leftover);
byteBuffer.position(pos); byteBuffer.position(pos);
LOG.warn("Leftover bytes when parsing message. Bytes: {}, complete message: {}", GB.hexdump(leftover), GB.hexdump(byteBuffer.array())); LOG.warn("Leftover bytes when parsing message type {}. Bytes: {}, complete message: {}", messageType, GB.hexdump(leftover), GB.hexdump(byteBuffer.array()));
} }
} }
} }