-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi,
Not really a great patch, but just a little change fixing the number of
unnecessary times `payload' getting defined in shell.c
Regards
Sudharshan S
blog: http://www.sudharsh.wordpress.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFHtX/9SsRjJNRMc4kRAjIOAJ9F04QpjcdYtBiMsv6YMsS/+zxRywCfe9Tf
Gqx5CYAV4p+0vVkd4A/8L20=
=W4Iv
-----END PGP SIGNATURE-----
Index: src/util/shell.c
===================================================================
--- src/util/shell.c (revision 4067)
+++ src/util/shell.c (working copy)
@@ -392,28 +392,23 @@
static int phone_msghandler(struct lgsm_handle *lh, struct gsmd_msg_hdr *gmh)
{
- char *payload;
+ char *payload = (char *)gmh + sizeof(*gmh);
int *intresult = (void *)gmh + sizeof(*gmh);
switch (gmh->msg_subtype) {
case GSMD_PHONE_GET_IMSI:
- payload = (char *)gmh + sizeof(*gmh);
printf("imsi <%s>\n", payload);
break;
case GSMD_PHONE_GET_MANUF:
- payload = (char *)gmh + sizeof(*gmh);
printf("manufacturer: %s\n", payload);
break;
case GSMD_PHONE_GET_MODEL:
- payload = (char *)gmh + sizeof(*gmh);
printf("model: %s\n", payload);
break;
case GSMD_PHONE_GET_REVISION:
- payload = (char *)gmh + sizeof(*gmh);
printf("revision: %s\n", payload);
break;
case GSMD_PHONE_GET_SERIAL:
- payload = (char *)gmh + sizeof(*gmh);
printf("serial: %s\n", payload);
break;
case GSMD_PHONE_POWERUP:
