README.devices: the PRINT button must get pressed for Metrix MX56C

srzip: don't access non-existing option

The srzip output module dropped support for the "filename" option in
commit 37875f750614 on 2015-07-30, but still used to assign to slot
options[0] which clobbers the array's sentinel. Remove those accesses
to the non-existing option.

hantek-4032l: Fix a compiler warning.

  api.c: In function 'config_set':
  api.c:352:5: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'uint64_t' [-Wformat=]
       sr_err("invalid sample range 2k...64M: %ld",
       ^

beaglelogic: Fix a compiler warning.

 beaglelogic_tcp.c: In function 'beaglelogic_tcp_send_cmd':
 beaglelogic_tcp.c:114:3: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat=]
     sr_dbg("Only sent %d/%lu bytes of command: '%s'.", out,
     ^

yokogawa-dlm: fix several compiler warnings (assignment, memory)

Check pointers' validity before dereferencing them. Release partially
allocated memory in an error path. Remove an assignment which never took
effect.

This was reported by clang's scan-build.

tondaj-sl-814: fix a potential memory leak

Release an allocated device instance in an error path.

This was reported by clang's scan-build.

sysclk-lwla: silence potential NULL dereference compiler warning

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

scpi-pps: silence potential NULL dereference compiler warning

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

saleae-logic-pro: silence "assigned, not used" compiler warning

Remove an assignment that never took effect.

This was reported by clang's scan-build.

rigol-ds: silence potential NULL dereference compiler warning

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

ols, pipistrello-ols: silence "assigned, not used" compiler warning

Each code path either assigns to 'flags', or leaves the routine. There
is no potential path that leaves the variable at the initially assigned
value, so the assignment took no effect. Remove it.

Nit: Trim the source code line length while we are here.

This was reported by clang's scan-build.

norma-dmm: fix a potential memory leak

Release an allocated buffer in an error path.

This was reported by clang's scan-build.

motech-lps-30x: fix several compiler warnings

Check pointers' validity before dereferencing them. Explicitly assign a
default value to variables, before conversion routines conditionally
assign the "real" value (and don't in case of conversion errors). This
avoids processing "garbage" data.

Strictly speaking I cannot see how the conversion routine returns OK and
has _not_ assigned a result. But the explicit assignment won't harm
either, and matches the fallback when the conversion fails (detectibly).
Which means that runtime behaviour won't change.

This was reported by clang's scan-build.

maynuo-m97: fix potential NULL dereference

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

manson-hcs-3xxx: fix potential NULL dereference

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

lecroy-xstream: fix several compiler warnings (assignment, memory)

Remove an assignment that won't take effect. Check pointers' validity
before dereferencing. Fix a memory leak.

This was reported by clang's scan-build.

lascar-el-usb: fix potential NULL dereference and memory leak

Check for successful allocation of multiple memory ranges, and release
partial allocations in the error path when one of them failed. This
fixes a potential memory leak, as well as avoids NULL dereferences.

This was reported by clang's scan-build.

korad-kaxxxxp: fix potential NULL dereference

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

hp-3457a: fix potential NULL dereference

Check the active channels' list for not being empty, before
dereferencing pointers.

This was reported by clang's scan-build.

hantek-6xxx: fix potential NULL dereference

Check pointers' validity before dereferencing.

This was reported by clang's scan-build.

hameg-hmo: fix several compiler warnings (assignments, memory)

Silence warnings about assigned values that never get used, potential
NULL deference, and potential memory leaks.

This was reported by clang's scan-build.

gmc-mh-1x-2x: fix potential memory leak

The scan_2x_bd232() routine used to always prepare one spare context,
and filled it in when a device was found, just to allocate another one
and continue scanning.

Free the last allocated context unconditionally, as it was allocated
unconditionally, and never used.

This was reported by clang's scan-build.

fx2lafw: silence "NULL dereference" compiler warning

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

ftdi-la: silence "assigned but never used" compiler warning

Stick with the prepared but unfinished "limit msec" code path, but
silence the warning about an unused variable value.

This was reported by clang's scan-build.

dreamsourcelab-dslogic: avoid division by zero

Do not align to block_size when its value is zero.

This was reported by clang's scan-build.

dreamsourcelab-dslogic: avoid NULL pointer dereference

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

chronovu-la: avoid NULL pointer dereference

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

beaglelogic: fixup use of uninitialized data in TCP routines

Make sure that failure to communicate via TCP results in access to
uninitialized data.

This was reported by clang's scan-build.

beaglelogic: fixup memory leak in the TCP command send routine

Release allocated memory in an error code path.

This was reported by clang's scan-build.

atten-pps3xxx: silence NULL dereference compiler warnings

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

session_driver: avoid division by zero, catch API violation

Avoid a division by zero, by not using a zero unitsize in a modulo
operation. As a byproduct, avoid processing and counting input that
neither has analog nor logic data. This should never have happened,
but the change now catches the error if invalid input is seen.

This was reported by clang's scan-build.

session: fixup access to uninitialized memory

The sr_packet_copy() routine could have written to an arbitrary memory
location. Make sure to allocate the space before writing to it, and
check for successful allocation before accessing the memory.

It's assumed that this error never took effect, as the routine appears
to be unused.

This was reported by clang's scan-build.

scpi: avoid uninitialized use of a variable

The 'opc' variable was only conditionally assigned to (depends on
successful SCPI communication). Ensure there always is a known value.

This was reported by clang's scan-build.

output/srzip: silence "use after free" compiler warning

Defer memory g_free() until after the data gets accessed one last time.

This was reported by clang's scan-build.

output/csv: silence NULL dereference compiler warnings

Check pointers' validity before dereferencing them.

This was reported by clang's scan-build.

input/raw_analog: silence "use after free" compiler warning

The cleanup() call will void the memory which 'inc' points to. Move the
dereference before the release.

This was reported by clang's scan-build.

lecroy-xstream: Wait for trigger before acquiring additional frames

lecroy-xstream: Properly handle a stopping acquisition

lecroy-xstream: Keep acquiring infinitely if no frame limit is set

hantek-dso: Capture ratio is a uint64_t.

From src/hwdriver.c:

  {SR_CONF_CAPTURE_RATIO, SR_T_UINT64, "captureratio",
          "Pre-trigger capture ratio", NULL},

hantek-dso: config_set(): Shorten SR_CONF_CAPTURE_RATIO case.

The sr_config_set() wrapper already checks for SR_CONF_CAPTURE_RATIO
being 0..100.

hantek-dso: Add missing SR_PRIV.

hantek-dso: Sample rate is a uint64_t.

Also make 'base' uint64_t, otherwise there'll be compiler warnings.

hantek-dso: Random cosmetics, drop unneeded debug output.

hantek-dso: dso2250: Fix capture runaway, only do the requested number of frames.

After the first capture ->num_frames never got to be _equal_
to ->limit_frames; fixed by resetting to zero in dev_acquisition_stop(),
and protected against similar problems in the future by switching to
greater-or-equal instead.

hantek-dso: dso2250: Allow setting TRIGGER_LEVEL as well.

hantek-dso: Fix a memory leak.

hantek-dso: Fix segfault when accessing already free()d memory.

This fix was guessed from other drivers' code.

This fixes bug #458.

hantek-dso: dso2250: It's not HORIZ_TRIGGERPOS but CAPTURE_RATIO.

Seems a bit unfortunately named; if my current guess is right,
perhaps it should be
  :%s/SR_CONF_HORIZ_TRIGGERPOS/SR_CONF_HORIZ_DISPLAYOFFSET/g

hantek-dso: dso2250: Support sample rates correctly.

Fast mode not done yet, only 125MHz allowed right now.

hantek-dso: dso2250: Report the sample rate back.

So that cursor (and other) measurements and the time axis are correct.

hantek-dso: dso2250: Initial support for the Hantek DSO 2250.

Crashes after first acquiration, but at least it _does_ acquire data.

udev rules: Add Hantek 4032L IDs.

hantek-4032l: Add initial driver implementation.

[Note: This patch is basically a squashed version of the initial driver
commits by Andreas Zschunke <redacted>, two fixes by
Andrej Valek <redacted>, and various coding style / cosmetic
fixes by Uwe Hermann <redacted> to make the driver a lot more
consistent with the rest of the libsigrok code-base.]

hantek-4032l: Initial driver skeleton.

zeroplus-logic-cube: Add LAP-16032U

rigol-ds: Fix crash when fetching logic channels

When refactoring the code, some places where sr_scpi_send was replaced
by rigol_ds_config_set the first argument was not changed from sdi->conn
to sdi. Fix the remaining ones.

Fix: https://sigrok.org/bug/1073

Signed-off-by: Stefan Brüns <redacted>

zeroplus-logic-cube: Fix Logian-16L metadata.

According to a vendor software screenshot of a user on IRC, this
device is detected as having 200MHz max. samplerate and 128kBit
memory per channel.

zeroplus: Add support for Meilhaus Logian-16L.

Untested yet, but pretty likely to work.

sysclk-lwla: Fix a regression with (at least) the LWLA1034.

This regression was introduced in f1ba6b4b2c9a8ecf90bb31efb218752aa7e49d1a.

Due to how the sysclk-lwla driver does up to 3 open/close operations in
one dev_open() API callback we cannot rely on the sr_dev_open() and
sr_dev_close() wrappers setting the sdi->status variable in this case.

Tested on LWLA1034.

sysclk-lwla: Fix a segfault in dev_close().

sr_dev_close(): Drop unneeded variable.

input/vcd: fixup VCD timestamp to sigrok samplenum mapping

When processing of large VCD input files was spread across multiple
parse_contents() invocations, the resulting sigrok stream of sample data
had gaps in them and total timing was off. For instance 74ms of input
data were interpreted as spanning some 600ms or 300ms, depending on the
number of channels in the input stream.

Move the "previous timestamp" variable to the input module context. This
eliminates the inappropriate gaps and fixes the translation of VCD file
timestamps to sigrok sample numbers.

This fixes bug #1075.

output/csv: Fix a typo.

60-libsigrok.rules: Add note about sigrok-androidutils.

fx2lafw: Update comments listing supported devices.

60-libsigrok.rules: Add missing entries and comments.

zeroplus-logic-cube: Add LAP-C(16128+) USB VID/PID.

Untested, so we don't know for sure it'll work.

This fixes bug #1045.

hameg-hmo: Forward internal channel state to sigrok channel state

This fixes bug #883.

korad-kaxxxxp: Add support for Korad KD3005P

Makefile.am: Add/update udev rules files.

cleanup udev part of README.devices, reflect split rules

Remove vague statements from the README. On all current distributions,
the udev paths are identical, anyone deliberately deviating from the
defaults should be able to handle it by themselves.

Rules in /etc/udev/rules.d/ should only be used for customization, or
for locally built packages.

contrib: Split device access policy from the device database

Split the distribution independent database from the access policy rules.
This avoids warnings due to granting permissions to the commonly unknown
plugdev group, and allows simple overrides of the used access policy.

contrib: remove duplicate rules for fx2lafw DSOs

contrib: rename udev rules for correct rule ordering on systemd systems

The "uaccess" tag has to be added before the "seat" rule is evaluated.
The upstream default for the seat rule is "71-seat.rules", so use
60-libsigrok.rules for appropriate lexicographical sorting.

Also use a dash instead of underscore, the latter is commonly used as a
replacement character of unsafe characters in autogenerated identifiers.

This fixes bug #1059.

scpi-pps: Channel group device options for HP 663xx

drivers: Fix locale dependent string to float conversion

Some drivers used locale dependent functions for converting strings
to float/double values. These functions fail when the decimal mark
is a "," in the locale settings but the string contains a ".".

This fixes bug #1064.

output/analog: Display META packets

arachnid-labs-re-load-pro: Add SR_DF_META packets for changed values/states

arachnid-labs-re-load-pro: Stop monitoring when open/close device

conrad-digi-35-cpu: Add SR_CONF_LIST for voltage_target and current_limit. Change current_limit lower limit.

lipgpib: Enhanced error message

demo: Properly handle low samplerates

A "low samplerate" in this case means anything where
samples_todo is less than SAMPLES_PER_FRAME. This case
wasn't handled properly before, resulting in wrong
amounts of data being sent out.

lecroy-xstream: Fix sample rate

We can't use the memory size command because it returns
the *maximum* memory used, not the *actual* memory used.
Hence, we only know the number of samples per div once
sample data actually comes in.

lecroy-xstream: Perform acquisition if no data is available

lecroy-xstream: Comment/style fixes

hantek-6xxx: use lower MAX_PACKET_SIZE on Windows

WinUSB seems to have a maximum size of 2M, reads of a larger size fail.

This fixes bug #1048.

lecroy-xstream: Implement config_channel_set API callback

lecroy-xstream: Fix trigger source/slope

lecroy-xstream: Random whitespace/style fixes

lecroy-xstream: Use best-effort strategy for unknown models

As there is a huge range of supported LeCroy scopes, naming
the IDN response for every single one of them is going to be
impossible. Hence, it makes more sense to treat all LeCroy
devices as if they were scopes and supported. This approach
lets users try to see how far they get and if they run into
issues, they can then be treated separately - e.g. by creating
a custom device profile.
Unless we do this, the vast majority of LeCroy scopes will
not be recognized by the driver.

lecroy-xstream: Fix COMM_HEADER and COMM_FORMAT

SCPI: Ignore IDN header in *IDN? response

Demo: Implement multi-frame development feature

Add std_session_send_frame_begin/end helpers

dreamsourcelab-dslogic: Fixed deinterleaving when non-contiguous set of channels are enabled

beaglelogic: Inline beaglelogic_devc_alloc().

beaglelogic: Minor whitespace and consistency fixes.

beaglelogic: No need to check g_free() argument.

glib's g_free() function will gracefully handle NULL as input.

beaglelogic: Coding style fixes

In beaglelogic_native.c and beaglelogic_tcp.c

Signed-off-by: Kumar Abhishek <redacted>