ACK: [PATCH] fwts_log: make line_number to a uint32_t type

Alex Hung alex.hung at canonical.com
Thu Sep 7 01:46:32 UTC 2017


On 2017-09-06 02:39 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Line numbers are never negative, so make line_number a uint32_t type
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/lib/include/fwts_log.h       | 2 +-
>   src/lib/src/fwts_log_plaintext.c | 3 ++-
>   src/lib/src/fwts_log_xml.c       | 3 ++-
>   3 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/src/lib/include/fwts_log.h b/src/lib/include/fwts_log.h
> index da8cb42b..3d4ecf9a 100644
> --- a/src/lib/include/fwts_log.h
> +++ b/src/lib/include/fwts_log.h
> @@ -97,7 +97,7 @@ typedef struct log_t {
>    */
>   typedef struct {
>   	FILE *fp;				/* file descriptor for log */
> -	int line_number;			/* keeps track of the line numbering */
> +	uint32_t line_number;			/* keeps track of the line numbering */
>   	fwts_log *log;				/* parent log struct */
>   	fwts_log_type type;			/* log type */
>   	fwts_log_filename_type filename_type;	/* log filename type */
> diff --git a/src/lib/src/fwts_log_plaintext.c b/src/lib/src/fwts_log_plaintext.c
> index a4fbcec4..0cde841f 100644
> --- a/src/lib/src/fwts_log_plaintext.c
> +++ b/src/lib/src/fwts_log_plaintext.c
> @@ -21,6 +21,7 @@
>   #include <stdlib.h>
>   #include <stdio.h>
>   #include <stdarg.h>
> +#include <stdint.h>
>   #include <string.h>
>   #include <unistd.h>
>   #include <sys/ioctl.h>
> @@ -52,7 +53,7 @@ static int fwts_log_header_plaintext(
>   			ptr++;
>   			if (!strncmp(ptr, "line", 4)) {
>   				n += snprintf(buffer + n, len - n,
> -					"%5.5d", log_file->line_number);
> +					"%5.5" PRIu32, log_file->line_number);
>   				ptr += 4;
>   			}
>   			if (!strncmp(ptr, "date", 4)) {
> diff --git a/src/lib/src/fwts_log_xml.c b/src/lib/src/fwts_log_xml.c
> index 273f0500..2954bb24 100644
> --- a/src/lib/src/fwts_log_xml.c
> +++ b/src/lib/src/fwts_log_xml.c
> @@ -20,6 +20,7 @@
>   #include <stdlib.h>
>   #include <stdio.h>
>   #include <stdarg.h>
> +#include <stdint.h>
>   #include <string.h>
>   #include <unistd.h>
>   #include <sys/ioctl.h>
> @@ -67,7 +68,7 @@ static int fwts_log_print_xml(
>   
>   	fprintf(log_file->fp, "%*s<logentry>\n", xml_stack_index * XML_INDENT, "");
>   
> -	fprintf(log_file->fp, "%*s<line_num>%d</line_num>\n",
> +	fprintf(log_file->fp, "%*s<line_num>%" PRIu32 "</line_num>\n",
>   		(xml_stack_index + 1) * XML_INDENT,
>   		"", log_file->line_number);
>   
> 
Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list