Restore printing of values in omshell to the style pre 21518. For

21518 we changed the print routines to always display time values
as a hex list.  This had a side effect of printing all data strings
as a hex list.  We shall investigate other ways of displaying time
values more usefully.  [ISC-Bugs #22626]

diff --git a/RELNOTES b/RELNOTES
index cc68e05e06fc2d8bbfd1ce3fb5eeb4b9e2e28777..c8e1a0e16947813ad18b327ce367948598686741 100644 (file)
--- a/RELNOTES
+++ b/RELNOTES
@@ -144,6 +144,11 @@ work on other platforms. Please report any problems and suggested fixes to
   a setsockopt() call.  The signal is already being ignored as part
   of the ISC library.  [ISC-Bugs #22269]
 
+- Restore printing of values in omshell to the style pre 21518.  For
+  21518 we changed the print routines to always display time values
+  as a hex list.  This had a side effect of printing all data strings
+  as a hex list.  We shall investigate other ways of displaying time
+  values more usefully.  [ISC-Bugs #22626]
 
                        Changes since 4.2.0b2
 

diff --git a/dhcpctl/omshell.c b/dhcpctl/omshell.c
index c3e7ef63dee354c5cd1904ee57ef0092e4133c31..4da7428e971191bd7b44e5943c15d6c73bfa9bfe 100644 (file)
--- a/dhcpctl/omshell.c
+++ b/dhcpctl/omshell.c
@@ -161,9 +161,9 @@ main(int argc, char **argv) {
                            break;
                                
                          case omapi_datatype_data:
-                           print_hex_only(v->value->u.buffer.len,
-                                          v->value->u.buffer.value,
-                                          sizeof(hex_buf), hex_buf);
+                           print_hex_or_string(v->value->u.buffer.len,
+                                               v->value->u.buffer.value,
+                                               sizeof(hex_buf), hex_buf);
                            printf("%s\n", hex_buf);
                            break;
                            

diff --git a/includes/dhcpd.h b/includes/dhcpd.h
index 3df59b0b90023f6fae78dca7da64f76e1018ab68..a096f1250afa64d1ac5398ab657b9417dc68ff31 100644 (file)
--- a/includes/dhcpd.h
+++ b/includes/dhcpd.h
@@ -2304,6 +2304,7 @@ void dump_packet PROTO ((struct packet *));
 void hash_dump PROTO ((struct hash_table *));
 char *print_hex PROTO ((unsigned, const u_int8_t *, unsigned, unsigned));
 void print_hex_only PROTO ((unsigned, const u_int8_t *, unsigned, char *));
+void print_hex_or_string PROTO ((unsigned, const u_int8_t *, unsigned, char *));
 #define print_hex_1(len, data, limit) print_hex(len, data, limit, 0)
 #define print_hex_2(len, data, limit) print_hex(len, data, limit, 1)
 #define print_hex_3(len, data, limit) print_hex(len, data, limit, 2)