Merge pull request #1120 from divine/patch-1

ehuss · web-flow · commit a3b90cb3756c · 2025-01-22T22:56:14.000Z
fix: prevent crash when repository url is empty
diff --git a/src/remote.rs b/src/remote.rs
@@ -147,7 +147,7 @@ impl<'repo> Remote<'repo> {
 
     /// Get the remote's URL as a byte array.
     pub fn url_bytes(&self) -> &[u8] {
-        unsafe { crate::opt_bytes(self, raw::git_remote_url(&*self.raw)).unwrap() }
+        unsafe { crate::opt_bytes(self, raw::git_remote_url(&*self.raw)).unwrap_or(&[]) }
     }
 
     /// Get the remote's pushurl.

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ impl<'repo> Remote<'repo> {`
`147`	`147`
`148`	`148`	`/// Get the remote's URL as a byte array.`
`149`	`149`	`pub fn url_bytes(&self) -> &[u8] {`
`150`		`- unsafe { crate::opt_bytes(self, raw::git_remote_url(&*self.raw)).unwrap() }`
	`150`	`+ unsafe { crate::opt_bytes(self, raw::git_remote_url(&*self.raw)).unwrap_or(&[]) }`
`151`	`151`	`}`
`152`	`152`
`153`	`153`	`/// Get the remote's pushurl.`