On Thu, Jul 14 2016, NeilBrown wrote: > If getaddrinfo() returns EAI_AGAIN, we shouldn't just give up, but > should continue normal retries as the nameserver may be unavailable > for the same reason as the NFS server. > > So move the getaddrinfo() call from nfs_validate_options() into > nfs_try_mounts() which is always called soon after, except in the > 'remount' case when we don't want it anyway. > > If EAI_AGAIN is returned, set errno to EAGAIN and allow this to be a > temporary failure. Otherwise report error and set errno to EALREADY > so no further message is given. > > Signed-off-by: NeilBrown <neilb@xxxxxxxx> > + > + if (!nfs_append_addr_option(mi->address->ai_addr, > + mi->address->ai_addrlen, mi->options)) > + return 0; > + mi->address = address; This patch is badly buggy. I'm de-referencing mi->address before assigning it. How did that ever pass quality control I ask myself. (fortunately SUSE does better QA than I do). I'll submit a revised version ... maybe put all the patches affecting the mountpoint export option at the end of the list. NeilBrown
Attachment:
signature.asc
Description: PGP signature
