diff options
| -rw-r--r-- | xpp/mpp.h | 2 | ||||
| -rw-r--r-- | xpp/mpptalk.c | 16 | ||||
| -rw-r--r-- | xpp/mpptalk_defs.h | 2 |
3 files changed, 18 insertions, 2 deletions
@@ -83,7 +83,7 @@ struct capkey { } PACKED; struct extrainfo { - char text[24]; + char text[EXTRAINFO_SIZE]; } PACKED; struct mpp_header { diff --git a/xpp/mpptalk.c b/xpp/mpptalk.c index a8bf12a..6b33743 100644 --- a/xpp/mpptalk.c +++ b/xpp/mpptalk.c @@ -366,7 +366,17 @@ int mpp_extrainfo_get(struct astribank_device *astribank, struct extrainfo *info } assert(reply->header.op == MPP_EXTRAINFO_GET_REPLY); if(info) { + int i; + memcpy(info, (void *)&CMD_FIELD(reply, MPP, EXTRAINFO_GET_REPLY, info), sizeof(*info)); + /* + * clean non-printing characters + */ + for (i = sizeof(*info) - 1; i >= 0; i--) { + if (info->text[i] != (char)0xFF) + break; + info->text[i] = '\0'; + } } free_command(reply); return 0; @@ -876,7 +886,11 @@ void show_astribank_status(struct astribank_device *astribank, FILE *fp) void show_extrainfo(const struct extrainfo *extrainfo, FILE *fp) { - fprintf(fp, "Extrainfo: : %s\n", (const char *)(extrainfo->text)); + char buf[EXTRAINFO_SIZE + 1]; + + memcpy(buf, extrainfo->text, EXTRAINFO_SIZE); + buf[EXTRAINFO_SIZE] = '\0'; /* assure null termination */ + fprintf(fp, "Extrainfo: : '%s'\n", buf); } int twinstar_show(struct astribank_device *astribank, FILE *fp) diff --git a/xpp/mpptalk_defs.h b/xpp/mpptalk_defs.h index e38f381..bc0b83b 100644 --- a/xpp/mpptalk_defs.h +++ b/xpp/mpptalk_defs.h @@ -108,4 +108,6 @@ enum dev_dest { DEST_EEPROM = 0x02, }; +#define EXTRAINFO_SIZE 24 + #endif /* MPPTALK_DEFS_H */ |