public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Maximiliano Sandoval <m.sandoval@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox] lang: Remove offsetof macro
Date: Mon, 26 Jan 2026 15:12:11 +0100	[thread overview]
Message-ID: <20260126141215.401447-1-m.sandoval@proxmox.com> (raw)

With rust 1.90 this will crash while running cargo test with

$ RUST_BACKTRACE=1 cargo test

null pointer dereference occurred
stack backtrace:
   0: __rustc::rust_begin_unwind
             at /usr/src/rustc-1.90.0/library/std/src/panicking.rs:697:5
   1: core::panicking::panic_nounwind_fmt::runtime
             at /usr/src/rustc-1.90.0/library/core/src/panicking.rs:117:22
   2: core::panicking::panic_nounwind_fmt
             at /usr/src/rustc-1.90.0/library/core/src/intrinsics/mod.rs:2367:9
   3: core::panicking::panic_null_pointer_dereference
             at /usr/src/rustc-1.90.0/library/core/src/panicking.rs:311:5
   4: rust_out::main::_doctest_main_proxmox_lang_src_lib_rs_71_0
   5: rust_out::main
   6: core::ops::function::FnOnce::call_once

Note that the deref_nullptr lint is going to be denied by default on 1.93.

Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
---

As far as my greeping was able to tell, there should be no other users than the
docstring.

 proxmox-lang/src/lib.rs | 22 ----------------------
 1 file changed, 22 deletions(-)

diff --git a/proxmox-lang/src/lib.rs b/proxmox-lang/src/lib.rs
index 0abfd507..0c33710b 100644
--- a/proxmox-lang/src/lib.rs
+++ b/proxmox-lang/src/lib.rs
@@ -66,28 +66,6 @@ macro_rules! static_assert_size {
     };
 }
 
-/// Evaluates to the offset (in bytes) of a given member within a struct
-///
-/// ```
-/// # use proxmox_lang::offsetof;
-///
-/// #[repr(C)]
-/// struct Stuff {
-///     first: u32,
-///     second: u32,
-/// }
-///
-/// assert_eq!(offsetof!(Stuff, second), 4);
-///
-/// ```
-#[deprecated = "use std::mem::offset_of! instead"]
-#[macro_export]
-macro_rules! offsetof {
-    ($ty:ty, $field:ident) => {
-        unsafe { &(*(std::ptr::null::<$ty>())).$field as *const _ as usize }
-    };
-}
-
 /// Shortcut for generating an `&'static CStr`.
 ///
 /// This takes a *string* (*not* a *byte-string*), appends a terminating zero, and calls
-- 
2.47.3



_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


             reply	other threads:[~2026-01-26 14:11 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-26 14:12 Maximiliano Sandoval [this message]
2026-01-30  8:15 ` Christian Ebner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260126141215.401447-1-m.sandoval@proxmox.com \
    --to=m.sandoval@proxmox.com \
    --cc=pbs-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal