- get_terminal_width_height does not and should never write to fd. Cosmetic Precaution

diff --git a/include/libbb.h b/include/libbb.h
index 30d3006..0b066d1 100644 (file)
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -570,7 +570,7 @@ extern void vfork_daemon(int nochdir, int noclose);
 extern void vfork_daemon_rexec(int nochdir, int noclose,
                int argc, char **argv, char *foreground_opt);
 #endif
-extern int get_terminal_width_height(int fd, int *width, int *height);
+extern int get_terminal_width_height(const int fd, int *width, int *height);
 
 int is_in_ino_dev_hashtable(const struct stat *statbuf, char **name);
 void add_to_ino_dev_hashtable(const struct stat *statbuf, const char *name);

diff --git a/libbb/xfuncs.c b/libbb/xfuncs.c
index dc160bf..240ac5d 100644 (file)
--- a/libbb/xfuncs.c
+++ b/libbb/xfuncs.c
@@ -572,7 +572,7 @@ void xstat(char *name, struct stat *stat_buf)
 
 /* It is perfectly ok to pass in a NULL for either width or for
  * height, in which case that value will not be set.  */
-int get_terminal_width_height(int fd, int *width, int *height)
+int get_terminal_width_height(const int fd, int *width, int *height)
 {
        struct winsize win = { 0, 0, 0, 0 };
        int ret = ioctl(fd, TIOCGWINSZ, &win);