sjg/trx-rs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[fix](trx-backend-soapysdr): remove silent fallback to first device when args are specified

Browse Source 
When specific SoapySDR device args are provided (e.g. with a serial number),
fail hard instead of silently falling back to Device::new("") which opens
the first available device. This caused multi-device setups to bind both
rig instances to the same physical device.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Stan Grams <sjg@haxx.space>
This commit is contained in:
Stan Grams
2026-03-21 15:23:06 +01:00
parent e6c34bb695
commit d74f77537a
1 changed files with 22 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/trx-server/trx-backend/trx-backend-soapysdr/src/real_iq_source.rs
+21 -27
View File
@@ -46,35 +46,29 @@ impl RealIqSource {
) -> Result<Self, String> {
tracing::info!("Initializing SoapySDR device with args: {}", args);
let device = match Device::new(args) {
Ok(dev) => dev,
Err(e) => {
tracing::warn!(
"Failed to open device with args '{}': {}. Attempting fallback...",
args,
e
);
match Device::new("") {
Ok(dev) => {
tracing::warn!(
"Successfully opened a device with empty args (fallback). \
Note: this may not be the intended device. \
If this is incorrect, check SoapySDR environment variables and plugins."
);
dev
}
Err(fallback_err) => {
return Err(format!(
"Failed to open SoapySDR device:\n \
Original args '{}': {}\n \
Fallback (empty args): {}\n \
let device = if args.trim().is_empty() {
// No specific device requested — open the first available device.
Device::new("").map_err(|e| {
format!(
"Failed to open SoapySDR device with empty args: {}\n \
Troubleshooting: Check that SoapySDR is installed and plugins are loaded. \
Try running SoapySDRUtil --probe to verify device availability.",
args, e, fallback_err
));
}
}
}
e
)
})?
} else {
// Specific device args provided (e.g. "driver=rtlsdr,serial=00000001").
// Do NOT fall back to empty args — the user explicitly requested a
// particular device and silently opening a different one would cause
// incorrect behaviour (especially with multiple devices).
Device::new(args).map_err(|e| {
format!(
"Failed to open SoapySDR device with args '{}': {}\n \
Troubleshooting: Verify the device is connected and the args are correct. \
Run SoapySDRUtil --find to list available devices and their identifiers.",
args, e
)
})?
};
tracing::info!("SoapySDR device opened successfully");