mirrors/lnd
1
0
Fork 0
mirror of https://github.com/lightningnetwork/lnd.git synced 2025-03-13 11:09:23 +01:00
Code Issues Projects Releases Wiki Activity

cmd+docs: fix cap out of range error in profile command

Browse source 
If `lncli profile remove` is called when there are no existing profiles,
it will cause a "cap out of range" panic. So in this commit, a length
check is added to avoid this.
This commit is contained in:
Elle Mouton 2022-08-31 15:31:49 +02:00
parent 9f013f5058
commit 96021fe85b
No known key found for this signature in database
GPG key ID: D7D916376026F177
2 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
cmd/lncli/cmd_profile.go
View file

@ -199,6 +199,10 @@ func profileRemove(ctx *cli.Context) error {
return fmt.Errorf("name argument missing")
}
if len(f.Profiles) == 0 {
return fmt.Errorf("there are no existing profiles")
}
// Create a copy of all profiles but don't include the one to delete.
newProfiles := make([]*profileEntry, 0, len(f.Profiles)-1)
for _, p := range f.Profiles {

3
docs/release-notes/release-notes-0.16.0.md
View file

@ -57,6 +57,9 @@ minimum version needed to build the project.
* [Fix](https://github.com/lightningnetwork/lnd/pull/6858) command line argument
parsing for `lncli sendpayment`.
* [Fix](https://github.com/lightningnetwork/lnd/pull/6875) mapslice cap out of
range error that occurs if the number of profiles is zero.
## Code Health
* [test: use `T.TempDir` to create temporary test