device: introduce routines to compare channels and channel lists

Applications are not prepared to handle changes in the channel list
between multiple acquisitions from the same source (device drivers
or input modules). Introduce common helpers to compare channels and
channel lists.

device: introduce common sr_channel_free() support code

There was the sr_channel_new() allocation routine, but releasing that
allocation was open-coded in call sites. Add the sr_channel_free()
routine for code re-use and consistency.

input/vcd: Expand the reset() logic

This addresses part of bug #1306. The reset() method of the VCD input
module was incomplete, and did not process new data upon second read.
Improve robustness and add missing reset instructions. Void invalid
pointers and avoid NULL dereferences in cleanup paths.

korad-kaxxxxp: add yet another KD3005P identification string

Apparently there are devices which identify as "KORADKD3005PV2.0" (no
whitespace, no trailing 0x01). Add another model entry.

Reported-By: Lars Pötter

zeroplus-logic-cube: Only emit log message upon unexpected number of bytes.

Fix various gcc 8 compiler warnings related to ARRAY_SIZE.

Example:

  In file included from src/hardware/kecheng-kc-330b/protocol.h:26,
                   from src/hardware/kecheng-kc-330b/api.c:22:
  src/hardware/kecheng-kc-330b/api.c: In function ‘config_list’:
  src/libsigrok-internal.h:51:34: warning: division ‘sizeof (void *) / sizeof (void)’ does not compute the number of array elements [-Wsizeof-pointer-div]
   #define ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
                                    ^
  src/libsigrok-internal.h:55:32: note: in expansion of macro ‘ARRAY_SIZE’
   #define ARRAY_AND_SIZE(a) (a), ARRAY_SIZE(a)
                                  ^~~~~~~~~~
  src/libsigrok-internal.h:964:43: note: in expansion of macro ‘ARRAY_AND_SIZE’
    std_opts_config_list(key, data, sdi, cg, ARRAY_AND_SIZE(scanopts), \
                                             ^~~~~~~~~~~~~~
  src/hardware/kecheng-kc-330b/api.c:296:10: note: in expansion of macro ‘STD_CONFIG_LIST’
     return STD_CONFIG_LIST(key, data, sdi, cg, NULL, drvopts, devopts);
            ^~~~~~~~~~~~~~~

fx2lafw: Add new VID:PID for usb-c-grok

serial-dmm: introduce support for subdriver specific channel names

Default to the existing "P1" etc naming scheme for analog channels of
serial-dmm subdrivers. Add support for subdriver specific channel names
(which can reference the channel number if they desire). This is useful
for devices with multiple displays, or special purpose devices where
other names than P1 can better reflect the channel's nature.

serial-dmm: only send acquisition data for enabled channels

Respect the user's "channel enabled" status. Do not feed the session bus
when data for disabled channels was received.

serial-dmm: count analog DMM channels starting at 1

Commit 556a926d432dd introduced support for multiple displays in
subdrivers of serial-dmm, but also changed user visible channel numbers
to start from 0. Restore the previous behaviour, start counting from 1
which users may perceive as more natural (serial-dmm used to start at P1
in the past, scopes start with CH1 as well).

brymen-bm86x: avoid NULL deref when usb->devhdl does not exist

There are code paths where dev_close() tries to access a USB handle
which does not exist. This was observed with this command:

  $ sigrok-cli -d brymen-bm86x --scan

Makefile: cosmetics, alpha-sort src/dmm/ entries

hantek-6xxx: Hantek 6022BL: Add VID/PID 04b5:602a support.

On Windows, this device can either enumerate as 04b4:602a or 04b5:602a,
depending on which vendor driver is currently being used, so we have to
support both in the hantek-6xxx driver as well.

This fixes bug #1295.

raw_analog: Set appropriate precision digits for sample format.

This fixes parts of bug #950.

analog output: convert binary to digital digits of precision.

This fixes parts of bug #950.

Fix read past end of array in sr_analog_si_prefix_friendly.

In the case where the input unit was not in the array, the for loop would
complete, but the following test would then read past the end of the array
since 'i' would already have been incremented to the array size.

Spotted because unitless data was getting SI prefixes with no unit, though
this would not have been deterministically reproducible.

This fixes parts of bug #950.

scpi-pps: Add a missing "break" in config_get().

Make sr_packet_{copy,free} API calls public.

This fixes bug #1277.

scpi-pps: Fix broken channel selection code.

Fixes bug #1279.

scpi-pps: Use software sample and time limits.

scpi-pps: Don't block waiting for a value on capture stop.

The comment says "A requested value is certainly on the way", but the code no
longer works this way. The receive handler requests a value and blocks until
it is received. There is no value pending between receive handler calls, so
this code now only leads to a timeout.

Add udev rules for Rigol DP800 series.

output/csv: Disable the dedup option by default.

Having this on by default is surprising to most users, since csv is
not usually associated with having a VCD-like dedup property/feature.

std: Drop unneeded/duplicate log messages.

The sr_session_send() function will already log all of those anyway.

hantek-4032l: Fix default threshold value selection (FP workaround).

Take value from generated array of range instead of using a hard-coded value.

Signed-off-by: Andrej Valek <redacted>

std: Fix up non-zero FP value generation

input: improve robustness, avoid NULL dereference in sr_input_send()

Applications might pass NULL for the buffer, and input modules might
accept it (or just cope). Eliminate a potential NULL dereference in
the emission of diagnostics messages.

proto.h: move routine declaration to its respective group

sr_resourcepaths_get(): Drop confusing log message.

This is not really needed and can sometimes confuse users.

hantek-6xxx: Hantek 6022BE: Add VID/PID 04b5:6022 support.

On Windows, this device can either enumerate as 04b4:6022 or 04b5:6022,
depending on which vendor driver is currently being used, so we have to
support both in the hantek-6xxx driver as well.

This fixes bug #918.

Consistently use the _WIN32 #define.

The _WIN32 variant is available pretty much on all compilers, others
might not be. G_OS_WIN32 would probably be an equally well-suited
alternative, but for now we standardize on _WIN32.

  http://nadeausoftware.com/articles/2012/01/c_c_tip_how_use_compiler_predefined_macros_detect_operating_system#WindowswithCygwinPOSIX

ipdbg-la: Fix two compiler warnings on Windows.

This is happening because the send() and recv() functions
have different prototypes on POSIX and Windows. Using the casts
is required on Windows and doesn't hurt on POSIX systems.

  [...]/protocol.c: In function 'tcp_send':
  [...]/protocol.c:161:26: warning: pointer targets in passing argument 2 of 'send' differ in signedness [-Wpointer-sign]
    out = send(tcp->socket, buf, len, 0);
                            ^
  In file included from [...]/protocol.c:24:0:
  [...]/include/winsock2.h:997:34: note: expected 'const char *' but argument is of type 'const uint8_t * {aka const unsigned char *}'
     WINSOCK_API_LINKAGE int WSAAPI send(SOCKET s,const char *buf,int len,int flags);
                                    ^
  [...]/protocol.c: In function 'ipdbg_la_tcp_receive':
  [...]/protocol.c:201:32: warning: pointer targets in passing argument 2 of 'recv' differ in signedness [-Wpointer-sign]
      int len = recv(tcp->socket, buf, 1, 0);
                                  ^
  In file included from [...]/protocol.c:24:0:
  [...]/include/winsock2.h:992:34: note: expected 'char *' but argument is of type 'uint8_t * {aka unsigned char *}'
     WINSOCK_API_LINKAGE int WSAAPI recv(SOCKET s,char *buf,int len,int flags);
                                    ^

ipdbg-la: Fix data_available() implementation on Windows.

  [...]/protocol.c: In function 'data_available':
  [...]/protocol.c:73:38: error: 'bytes_available' undeclared (first use in this function)
    ioctlsocket(tcp->socket, FIONREAD, &bytes_available);
                                        ^
  [...]/protocol.c:73:38: note: each undeclared identifier is reported only once for each function it appears in
  [...]/protocol.c:84:1: warning: no return statement in function returning non-void [-Wreturn-type]
   }
   ^

ipdbg-la: Fix a build issue on Windows.

  [...]/protocol.c:41:23: fatal error: sys/ioctl.h: No such file or directory

fx2lafw: silence error message in query for channel group's device options

The fx2lafw(4) driver supports mere logic analyzers as well as mixed
signal devices, but does not support channel group specific device
options. Avoid an error message when channel group device options get
queried, the condition is perfectly legal and non-fatal.

How to reproduce:
  $ pulseview -d fx2lafw
  $ sigrok-cli -d fx2lafw -g Logic --show

This fixes bug #1267.

saleae and other FX2: show firmware name when loading fails

The "firmware load failed" message would be even more helpful if users
could learn which firmware file failed to load. Add those filenames to
various FX2-based drivers.

This addresses bug #1262.

Add a new sr_log_callback_get() API call.

ipdbg-la: scan(): Use g_strdup_printf().

ipdbg-la: Make some functions static.

ipdbg-la: Avoid CamelCaps and ALLCAPS variable names.

ipdbg-la: Drop some unneeded casts.

ipdbg-la: data_available(): Return gboolean.

ipdbg-la: Use std_init() and std_dev_list().

ipdbg-la: Consistently use g_strerror().

ipdbg-la: Drop unneeded g_malloc0() checks for small allocations.

ipdbg-la: Drop unneeded sdi->status handling.

This is already done by the backend wrapper functions.

ipdbg-la: Simplify config_get/_set/_list.

ipdbg-la: Fix devopts[] contents.

ipdbg-la: Drop various unneeded name prefixes.

ipdbg-la: Fix incorrect copyright headers.

As per Daniel Anselmi <redacted> in an email conversation, the
code was actually written by Eva Kissling <redacted>
(as indicated in the commit logs as well). Fix the headers accordingly.

ipdbg-la: Consistently use the same naming everywhere.

Use "ipdbg-la" everywhere to refer to the driver, including
in function name prefixes etc. There's no need to encode
website details (.org) into the driver/function name(s).

ipdbg-la: Allow rx to time out and handle invalid data properly

This is required for when we connect to invalid devices, e.g.
port 4243 of the IPDBG host instead of 4242.

ipdbg-la: Remove SR_CONF_SERIALCOMM

ipdbg-la: Adjust to sigrok indentation style

ipdbg-la: More style fixes

ipdbg-la: Style fixes

ipdbg-la: stop-command

ipdbg-la: stop-function pulseview

ipdbg-la: add edge trigger option

ipdbg-la: revers order of trigger settings

ipdbg-la: return correct samples limit

ipdbg-la: add "loggs"

ipdbg-la: get rid of crash on shutdown

ipdbg-la: reduce warnings (added explicit casts)

ipdbg-la: Add changes for IPDBG project

sr_buildinfo_libs_get(): Also support LIBUSBX_API_VERSION.

This fixes a build issue on some distros / libusb versions.

Fix multiple C++ bindings warnings with gcc 8.

  bindings/cxx/classes.cpp: In function ‘int sigrok::call_log_callback(void*, int, const char*, __va_list_tag*)’:
  bindings/cxx/classes.cpp:242:17: warning: catching polymorphic type ‘class sigrok::Error’ by value [-Wcatch-value=]
    } catch (Error e) {
                   ^
  In file included from bindings/cxx/classes.cpp:1667:
  bindings/cxx/enums.cpp: In static member function ‘static Glib::VariantBase sigrok::ConfigKey::parse_string(std::__cxx11::string, sr_datatype)’:
  bindings/cxx/enums.cpp:789:13: warning: catching polymorphic type ‘class std::invalid_argument’ by value [-Wcatch-value=]
      } catch (invalid_argument) {
               ^~~~~~~~~~~~~~~~
  bindings/cxx/enums.cpp:804:13: warning: catching polymorphic type ‘class std::invalid_argument’ by value [-Wcatch-value=]
      } catch (invalid_argument) {
               ^~~~~~~~~~~~~~~~

input/wav: Fix potential buffer overflow (and compiler warning).

With gcc 8 this yielded:

  src/input/wav.c: In function ‘receive’:
  src/input/wav.c:345:51: warning: ‘%d’ directive output may be truncated writing between 1 and 10 bytes into a region of size 6 [-Wformat-truncation=]
       snprintf(channelname, sizeof(channelname), "CH%d", i + 1);
                                                     ^~
  src/input/wav.c:345:48: note: directive argument in the range [1, 2147483647]
       snprintf(channelname, sizeof(channelname), "CH%d", i + 1);
                                                  ^~~~~~
  src/input/wav.c:345:5: note: ‘snprintf’ output between 4 and 13 bytes into a destination of size 8
       snprintf(channelname, sizeof(channelname), "CH%d", i + 1);
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

strutil.c: Use gboolean in favor of bool for consistency.

tests: add (negative) "." test case for rational conversion

A naive implementation of the parse rational conversion helper could
have accepted invalid input. Check for the expected conversion failure.

tests: add more cases for text to rational number conversion

Test the ".1" and "1." cases which are assumed to be problematic on
MacOS (or may not have been supported before a recent update). Add more
tests with leading signs as well as whitespace instead of a sign.

strutil: insist in some mantissa for parse rational

The previous implementation accepted either empty integer or empty
fractional parts of a floating point number, but also when both parts
were missing ("." input). Insist in at least one of the parts to be
present.

strutil: accept leading whitespace in parse rational

Programmatic output of floating point numbers might choose to print
spaces instead of an explicit '+' sign, to align the output with the
result of formatting negative numbers yet achieve a screen appearance
similar to what humans would have written. Skip leading whitespace
before insisting in seeing either signs or digits or decimals.

strutil: handle empty fractional in parse rational

Accept numbers like "123." where the period (dot) is present yet the
fractional part is empty. Adding a period but no additional digits is a
popular method of turning an otherwise integer literal into a float.
Compilers and strtod() routines accept this notation, too, so we have to
expect seeing such input.

strutil: cosmetics, fixup minor style nits

tests: echo text input when rational conversion fails

The previous implementation only echoed the p/q conversion results _if_
the return code signalled success but the result was unexpected. Although
the errno value for failed conversion attempts (non-zero return codes)
is not too helpful, seeing which text input failed the test is desirable.

input/vcd: abort VCD import when timestamp counts backwards

The VCD specification requests that timestamps will strictly increase as
one advances through the file. Add another check where the previous
implementation resulted in a tight loop and made the application stall.
Do print an error message and abort file processing in that case.

This fixes bug #1250.

sr_buildinfo_libs_get(): Show LIBUSB_API_VERSION.

configure.ac: Emit a warning if the C++ bindings are not being built.

strutil: Fix unit test failure (bug #1093).

Patch by Wojciech Lazarski, thanks a lot!

output: simplify trigger marker position calculation (readability)

This amends commit 67b345b981a5 which fixed the calculation of the
trigger marker's position. Improve readability of the formulae and
adjust comments.

output: fixup trigger marker position in ascii/bits/hex output modules

Adjust the calculation of the '^' marker's position in T: lines of the
-O ascii/bits/hex output modules such that it matches the sample data
lines' layout. Add comments which discuss the motivation of the marker
position's calculation, which differs among each of those modules.

Strictly speaking -O bits was already correct. But I chose to adjust and
comment the logic such that multiple output modules follow a common
pattern. If performance is an issue, the bits.c change might be worth
reverting.

This commit fixes bug #1238.

std: Improve prototypes of some functions.

rigol-ds: Add initial Agilent MSO7034A support.

Digital channels (and various other features) are not yet supported.

rigol-ds: Improve a log message.

siglent-sds: Drop currently unused switch/case.

siglent_sds_channel_start(): Use ternary operator to shorten code.

rigol-ds: Add missing "return SR_ERR".

siglent-sds: Fixed samplerate and memory depth calculation

siglent-sds: Fix, USB connection problem partially solved, bug #1130

udev: Add rule for Rigol DS4000 series devices.

rigol-ds: Initial patch for Rigol DS4000 support

This fixes bug #1208.

rigol-ds: Rename the CHANINFO macro and comment it

rigol-ds: Store trigger sources and their number for each model

Store trigger sources and their number for each model when it is probed
and use that.

This fixes bug #299.

tests: Expand test_analog_unit_to_string unit test.

tests: Fix test_analog_unit_to_string unit test.

The loop body was never executed.

serial-dmm: Fix Voltcraft VC-96 sorting.

dmm/vc96: Drop some obsolete debug output.

Add support for the Voltcraft VC-96 DMM.

dtm0660: Do not apply the exponent twice on the value

This fixes bug #1236.